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

Allow quantity to go beyond funds #194

Closed
wants to merge 261 commits into from
Closed

Allow quantity to go beyond funds #194

wants to merge 261 commits into from

Conversation

lauchness
Copy link
Collaborator

@lauchness lauchness commented Aug 10, 2023 •
edited
Loading

Description

This PR ensures the fundsStatus is only updated to "bridging" once per address. After that, the status will be "insufficient" which will allow users to modify quantity beyond their funding threshold, and will display the insufficient funds page when they make the action to mint.

Issue

Notion Ticket

ximxim and others added 30 commits August 7, 2023 17:11
@ximxim
docs: update readme file
c908114
@lauchness
Merge pull request #8 from LazerTechnologies/staging
1dcfc57 
Staging
@ximxim
Merge pull request #3 from LazerTechnologies/generate-tweets
ed9ca2b 
chore: using generate tweets instead of on the fly tweets fetch
@lauchness
Merge pull request #9 from LazerTechnologies/staging
3b2996e 
Staging
@DjordjeVujatovic
chore: teaser page layout cleanup
1c22b17
@ximxim
chore: added github workflow to commit empty message to trigger Verce…
6f3040f 
…l build under my name
@ximxim
chore: testing with repo url
2e9fe3d
@ximxim
chore: testing with repo owner logic
9523dd8
@ximxim
chore: synced schedule dates
83b6d9f
@DjordjeVujatovic
chore: update footer design + layout
61a499c
@lauchness
feat: replace next-image with img to avoid caching issues
fee06ec
@wilsoncusack
Merge pull request #10 from LazerTechnologies/allow-external-users-to…
71d6f4f 
…-trigger-vercel-build

chore: added github workflow to commit empty message to trigger build
@ximxim
empty commit
8bbe26d
@wilsoncusack
feat: restructure validation logic to support zora and external, exte…
90e5b59 
…rnal unfinished
@wilsoncusack
feat: show max per user mint in button message
1bdd3fa
@wilsoncusack
feat: refactor to enable zora write tx
68b5b99
@wilsoncusack
feat: allow zora native minting
dca219f
@wilsoncusack
fix: sync useValidate
7394a68
@wilsoncusack
feat: add zora test mint
7c39aa0
@wilsoncusack
feat: new crossmint id
daff830
@wilsoncusack
fix: crossmint working with zora
021a1be
@ximxim
empty commit
a73537b
@DjordjeVujatovic
chore: add links to footer
e112cdd
@DjordjeVujatovic
chore: undo mobile nav styles
f2e1249
@DjordjeVujatovic
chore: implement new mobile nav design
d22ba40
@arpitsrivastava-cb
fix: top margin for community
6982853 
Signed-off-by: Arpit Srivastava <[email protected]>
@ximxim
empty commit
80701f3
@arpitsrivastava-cb
fix: padding issue
95f83a6 
Signed-off-by: Arpit Srivastava <[email protected]>
@ximxim
empty commit
582c4ac
@lauchness
feat: remove console.log
fdc7379
ximxim and others added 23 commits August 9, 2023 17:44
@ximxim
empty commit
6c7537e
@ximxim
empty commit
96c1a56
@ximxim
chore: merged staging
40ec03d
@ximxim
chore: removed console log
c0cac5d
@ximxim
chore: password protect endpoint
7492bcf
@ximxim
Merge remote-tracking branch 'upstream/master' into master-test
910c3fe
@ximxim
Merge remote-tracking branch 'upstream/master'
47ac4e8
@ximxim
empty commit
f867c7a
@ximxim
Merge pull request #75 from LazerTechnologies/cache-mirror
bb298fd 
fix: mirror caching in file
@ximxim
Merge pull request #76 from LazerTechnologies/staging
b243459 
Staging
@DjordjeVujatovic
chore: update fwb description
3069fe0
@lauchness
feat: remove est time from getNow
afc499d
@lauchness
feat: fix spacing in partner hero
297f114
@lauchness
feat: fix tz date formatting for app/page and app/[slug]/page
27d1880
@lauchness
Merge pull request #78 from LazerTechnologies/partner-hero-spacing
0e8a17d 
feat: fix spacing in partner hero
@ximxim
Merge pull request #77 from LazerTechnologies/chore/update-fwb-descri…
f7d0c5c 
…ption

chore: update fwb description
@lauchness
Merge branch 'master' into fix-tx-issue
335e9a9
@lauchness
fix: format today string based on UTC values
8035d28
@lauchness
feat: get drop date formatting
24b8448
@lauchness
Merge pull request #79 from LazerTechnologies/fix-tx-issue
646ab33 
Fix timezone issue
@ximxim
chore: merged upstream master
bdc2d09
@lauchness
Merge branch 'master' of github.com:LazerTechnologies/onchainsummer.xyz
b5ccf09
@lauchness
fix: prevent checking for bridge more than once per address:
1875cba
@vercel
Copy link

vercel bot commented Aug 10, 2023 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
onchainsummer-xyz ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 10, 2023 11:46pm

@wilsoncusack
Copy link
Collaborator

closed in favor of #203

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DjordjeVujatovic DjordjeVujatovic Awaiting requested review from DjordjeVujatovic

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@lauchness @wilsoncusack @ximxim @DjordjeVujatovic @arpitsrivastava-cb @lukasrosario @AshishYelekar