Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

qa for deployer bug #1

Open
wants to merge 198 commits into
base: development
Choose a base branch
from
Open

qa for deployer bug #1

wants to merge 198 commits into from

Conversation

zugdev
Copy link
Owner

@zugdev zugdev commented Oct 14, 2024

Resolves #1

EresDev added 30 commits May 11, 2024 03:19
@EresDev
feat: claim permit in gift cards
90ee27f
@EresDev
refactor: move addresses to constants from env
5426f77
@EresDev
fix: compares owner address case insensitive
e0a787b
@EresDev
fix: deduct fee from reward amount
7a61714
@EresDev
fix: show unavailable range cards price correctly
abf145f
@EresDev
refactor: rename func to clear its effect
2aada64
@EresDev
fix: use correct pricing for gift card
52c0979
@EresDev
refactor: move pricing & duplicatre code to proper files/vars
f1d58c3
@EresDev
fix: offer only visa & mastercard
9ebe8f0
@EresDev
fix: add disclaimer below gift cards list
47e644e
@EresDev
docs: add info about API base url
56cba03
@EresDev
fix: allow only ERC20 permit to claim card
cf21902
@EresDev
fix: prevent large prices to take second line on cards list
f056b9d
@EresDev
chore: run missing yarn install from previous merge
ed6c28e
@EresDev
chore: handle cspell warnings by listing & fixing words
afe5f9e
@EresDev
refactor: replace word usage product with gift-card
84763f3
@EresDev
fix: check permit transfers correct erc20 token on claim gift card
8d18bba
@EresDev
fix: check permit token owner is correct on claim gift card
7cc3fc5
@EresDev
fix: validate correct token transfer on different chains
8fe9b2b
@EresDev
refactor: allow tests to generate permits without CLI
83b816a
@EresDev
fix: show price/value upto 2 decimals in title
bd17ddb
@EresDev
test: add tests for claiming gift cards
eff0545
@EresDev
fix: add date filter to retrieve old order
d12b9bb
@EresDev
test: ignore balance because it changes with tests
0e909ec
@EresDev
chore: fix spell
082f52b
@EresDev
fix: run wrangler in workflow for backend in tests
678ca44
@EresDev
fix: use reloadly sandbox by default
e28ac13
@EresDev
fix: log if using reloadly sandbox or not
5dee0f7
@EresDev
fix: change anvil RPC to fork for tests
c16253d 
Old RPC was causing issues randomly when forked with anvil
@EresDev
fix: start wrangler in action for tests backend
530f3d1
@github-actions GitHub Actions
Copy link

github-actions bot commented Oct 14, 2024 •
edited
Loading

Unused dependencies (2)

Filename dependencies
package.json lodash.capitalize
react

Repository owner deleted a comment from cloudflare-workers-and-pages bot Oct 14, 2024
@github-actions GitHub Actions
Copy link

github-actions bot commented Oct 15, 2024 •
edited
Loading

Preview Deployment
b0b3cf55f249ac2d9caea9815e6d44b2aee1dbfb

Repository owner deleted a comment from cloudflare-workers-and-pages bot Oct 15, 2024
@zugdev zugdev mentioned this pull request Oct 15, 2024
Merged
Repository owner deleted a comment from cloudflare-workers-and-pages bot Oct 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@zugdev @EresDev @0x4007 @Keyrxng @rndquu @whilefoo