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.

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

🎫[STAMPS] As a passport holder who has started the BrightID process, I want to see expectations to BrightID's process #128

Closed
brentmartin opened this issue Jun 1, 2022 · 4 comments · Fixed by #135
Assignees

Comments

@brentmartin
Copy link

brentmartin commented Jun 1, 2022

GIVEN that I have signed into my passport with a wallet address
WHEN I click the BrightID stamp
THEN I see a modal that informs me of BrightID's process and gives me expectation of when to return and verify
AND WHEN I click Connect to BrightID
THEN I see a notification that informs me verification expectations

@brentmartin
Copy link
Author

related to story #1

@brentmartin
Copy link
Author

brentmartin commented Jun 1, 2022

Changes to content on the modal:

ADD

BrightID is a social identity network that allows you to prove that you’re only using one account. Learn More.


CHANGE

A verifiable credential was not generated for your address. Follow the steps below to qualify:

to

To collect an Identity Stamp for BrightID, please complete the following steps.


KEEP

1) Download the BrightID App on your mobile device


CHANGE

2) Connect BrightID to Gitcoin by scanning this QR code from the BrightID app, or clicking here from your mobile device.

to

2) Link BrightID to Gitcoin by scanning this QR code from the BrightID app, or clicking here from your mobile device.


ADD

3) Click Connect BrightID to get sponsored by Gitcoin.

Once you are linked, sponsored, and have attended a connection party to complete your verification* on BrightID's App - return to this Stamp to finish the verification.

* Please note that it may take some time for BrightID to complete this process

@brentmartin
Copy link
Author

brentmartin commented Jun 1, 2022

Changes to content on the notification:

CHANGE

Success
Successfully triggered BrightID Sponsorship. Come back to Passport to Verify.

to

Sponsored through Gitcoin for BrightID
For verification status updates, check BrightID's App. Once you are verified by BrightID - return here to complete this Stamp.


And then apply a neutral color to the notification. Perhaps the purple we use elsewhere - #6f3ff5

@brentmartin brentmartin moved this from Backlog to In Progress (WIP) in Passport Jun 1, 2022
@farque65 farque65 self-assigned this Jun 2, 2022
farque65 added a commit that referenced this issue Jun 2, 2022

Verified

This commit was signed with the committer’s verified signature.
atodorov-storpool Anton Todorov
- adds custom toast style

Closes #128
farque65 added a commit that referenced this issue Jun 2, 2022

Verified

This commit was signed with the committer’s verified signature.
atodorov-storpool Anton Todorov
- adds custom toast style

Closes #128
farque65 added a commit that referenced this issue Jun 2, 2022
farque65 added a commit that referenced this issue Jun 2, 2022

Verified

This commit was signed with the committer’s verified signature.
rsmontero Ruben S. Montero
- adds custom toast style

Closes #128
Repository owner moved this from In Progress (WIP) to In Staging - Needs Review in Passport Jun 2, 2022
farque65 added a commit that referenced this issue Jun 2, 2022

Verified

This commit was signed with the committer’s verified signature.
atodorov-storpool Anton Todorov
- adds custom toast style

Closes #128
@brentmartin brentmartin moved this from In Staging - Needs Review to Tested/Accepted in Passport Jun 2, 2022
@brentmartin
Copy link
Author

AC looks great to me - for this and #1

0xflywill pushed a commit to 0xflywill/passport that referenced this issue Sep 6, 2022

Verified

This commit was signed with the committer’s verified signature.
atodorov-storpool Anton Todorov
- adds custom toast style

Closes passportxyz#128
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants