-
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-269 update user menu and profile edit sidebar with orders #2486
feat: APP-269 update user menu and profile edit sidebar with orders #2486
Conversation
✅ Deploy Preview for regen-website ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
web-marketplace/src/components/organisms/AdminNavigation/AdminNavigation.tsx
Outdated
Show resolved
Hide resolved
web-marketplace/src/components/organisms/AdminNavigation/AdminNavigation.tsx
Outdated
Show resolved
Hide resolved
b6e42bb
to
2ab767a
Compare
web-marketplace/src/components/organisms/RegistryLayout/RegistryLayout.Header.tsx
Outdated
Show resolved
Hide resolved
4104212
to
ceaaef4
Compare
...eries/react-query/registry-server/getPaymentMethodsQuery/getPaymentMethodsQuery.constants.ts
Outdated
Show resolved
Hide resolved
web-marketplace/src/components/organisms/RegistryLayout/RegistryLayout.Header.tsx
Outdated
Show resolved
Hide resolved
2feb15a
to
989910e
Compare
@erikalogie could you have a look at this? |
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.
LGTM
989910e
to
d6c8364
Compare
Looks like something is failing, let me know when it is back up to test |
@erikalogie this is ready for review. There was an issue between the new Accounts Orders page and this feature that required some more updates.
@blushi after these changes you may want to have a look again |
We discussed a few things about the layout on slack, but also let's do the "P" in "Edit profile" as lower case and the same with the O in "My orders" |
@erikalogie this is ready for another look |
c967926
to
6e6a8ad
Compare
@erikalogie can you have another look please? |
6e6a8ad
to
884644b
Compare
@erikalogie see latest update |
@r41ph I am having the weirdest thing happen. It feels like when I'm trying to scroll on these pages, like it is extra hard, kinda like my mouse is stuck. When I get down to the footer then it scrolls easily. |
FYI @r41ph before merging this, we should probably just hide the pages that aren't implemented yet, eg all except edit profile and settings |
63899ea
to
b250da1
Compare
Description
https://regennetwork.atlassian.net/browse/APP-269
Author Checklist
I have...
How to test
Reviewers 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...