8000 Finish zkSync recovery page by mds1 · Pull Request #7623 · gitcoinco/web · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Finish zkSync recovery page #7623

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
Merged
merged 1 commit into from
Oct 6, 2020
Merged

Finish zkSync recovery page #7623

merged 1 commit into from
Oct 6, 2020

Conversation

mds1
Copy link
Contributor
@mds1 mds1 commented Oct 6, 2020

Steps to test

  1. Checkout using zkSync + L1 transaction
  2. Close the page immediately after confirming your deposit tx
  3. Wait for it to be mined, then wait ~1-2 minutes for zkSync to recognize it (1 confirmation on Rinkeby, 10 on mainnet)
  4. Run docker-compose exec -e DJANGO_SETTINGS_MODULE="app.settings" web python3 app/manage.py update_tx_status. The validator comment for your contributions should now have either Transaction not found, unknown reason OR Found deposit but no transfer. This depends on the status of your zkSync account. Either one is ok
  5. Go to the /grants/zksync-recovery page
  6. Read the copy, use the button to send the rest of your transactions
  7. Page should redirect when complete and show a modal like the screenshot below (the undefined text is expected because we aren't using the user's actual cart data, but instead are pulling a fake, minimal cart from the database)
  8. Run docker-compose exec -e DJANGO_SETTINGS_MODULE="app.settings" web python3 app/manage.py update_tx_status. The contributions should now be marked as successful

image

@mds1 mds1 requested a review from thelostone-mc October 6, 2020 15:42
@thelostone-mc thelostone-mc merged commit 151302a into gitcoinco:stable Oct 6, 2020
@mds1 mds1 mentioned this pull request Mar 9, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@thelostone-mc thelostone-mc thelostone-mc approved these changes

@danlipert danlipert Awaiting requested review from danlipert

@frankchen07 frankchen07 Awaiting requested review from frankchen07

@octavioamu octavioamu Awaiting requested review from octavioamu

@PixelantDesign PixelantDesign Awaiting requested review from PixelantDesign

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

Successfully merging this pull request may close these issues.
2 participants
@mds1 @thelostone-mc
0