-
Notifications
You must be signed in to change notification settings - Fork 1
115 onboarding member yamada migration #119
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
Conversation
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.
Hi @mikais13 thanks for your work! I have left a few comments, please take a look!
You can take a look at the pages I've worked on like src/app/(frontend)/(dev)/account/page.tsx
for examples.
If you are using RadioCard
, I think you should also wrap it in <RadioCardGroup withIcon={false}>
from the documentation.
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.
Once you delete the Dockerfile I think it is good to merge! Good job!!
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 Nice!
Description
Replaced member page components and styling with equivalents in Yamada UI.
Before:



After:



Fixes #115
Type of change
How Has This Been Tested?
Checklist before requesting a review