-
Notifications
You must be signed in to change notification settings - Fork 74
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
Emc/fix/1944-profile-portfolio-issues #1945
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Ignored Deployment
|
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.
@0xemc which two issues does this PR fix?
…le-portfolio-issues
@jabby if you're ok with these fixes we can close this PR and tackle the others after. CC: @Atmosfearful |
@0xemc checking now |
I tested mainnet and testnet logins Only minor issue is when viewing your own page and logging in, there is a flash if missing listings while the React component is swapped from the public view to the SellerConnected view. It's a minor issue but I'm sure we can find a solution by passing props down. Merging and releasing. Thanks for staying up late for this one @0xemc |
Description
=== null
check that was failing because ofundefined
Resolves the following from #1944:
JABBY CONFIRMS FIXED
JABBY CONFIRMS FIXED
JABBY CONFIRMS FIXED