-
Notifications
You must be signed in to change notification settings - Fork 9
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
feat: APP-163 custodial wallet design implementation #2470
feat: APP-163 custodial wallet design implementation #2470
Conversation
✅ Deploy Preview for regen-website ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@erikalogie see testing instructions |
LGTM! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good, just one suggestion
web-storybook/.storybook/preview.jsx
Outdated
<QueryClientProvider client={reactQueryClient}> | ||
<AuthApolloProvider apolloClientFactory={apolloClientFactory}> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not sure if we should have storybook depend on those providers, which are really marketplace related
Instead if we want to keep the UserAccountSettings story, we should probably refactor it to not use those providers directly but rather provide what we need as props (from useEmailConfirmationData
)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@blushi updated as per our conversation, please have a look
0e659ec
to
29adbad
Compare
da1b6e2
to
74373e3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tACK
Description
https://regennetwork.atlassian.net/browse/APP-163
This PR also fixes an error with missing providers in the related stories in Storybook: https://regen-storybook.netlify.app/?path=/story/registry-organisms-useraccountsettings--default
Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...
How to test
Note: theLearn more»
link needs to be updated with the correct URLReviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...