Connect storefront to backend and add wallet component. #873
Conversation
jorgelo
commented
Feb 6, 2019
•
jorgelo
commented
- Adds logic to redirect to connect your wallet.
- Includes Toby's add wallet component.
- Covers tickets CIVIL-307 and CIVIL-310
- Adds a nice little helper called LoadUser that returns the user if there is one.
…ct-storefront-to-backend
…storefront-to-backend
…onnect-storefront-to-backend
…onnect-storefront-to-backend
…onnect-storefront-to-backend
…hub.com:joincivil/Civil into jorgelo/CIVIL-307/connect-storefront-to-backend
| pathname: `/tokens`, | ||
| state: {}, | ||
| }); | ||
| // TODO(jorgelo): Same as above, there should be a better way to force a refetch. |
There was a problem hiding this comment.
Check this again after we merge my PR #867 with updated AccountEthAuth component - the update property passed to <Mutation> updates the cached value for the currentUser { ethAddress } query, which I damn hope also updates the cached value for the larger currentUser query you use, so it should update state without refresh.
| return ( | ||
| <Query query={userQuery}> | ||
| {({ loading, error, data }) => { | ||
| if (loading || error) { |
There was a problem hiding this comment.
If you haven't already you should double check that this handles the logged-out state (which will cause this request to error) in the way you expect - unless you're only using this component in routes that require authentication?
| return ( | ||
| <TokenAccountOuter> | ||
| <TokenAccountInner> | ||
| <UserTokenAccountHeader /> | ||
| <FlexColumns> | ||
| <FlexColumnsPrimary> | ||
| <FlexColumnsPrimaryModule> | ||
| <UserTokenAccountSignup step={"completed"} /> | ||
| <UserTokenAccountSignup step={accountSignupStep} /> |
There was a problem hiding this comment.
I updated this flow a little bit in a new branch so we're going to have some merge conflicts to work out. Should be easy fixes but there are a few things we should discuss. Same with TokensAccountSignup.tsx
