-
Notifications
You must be signed in to change notification settings - Fork 0
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
Update UI #135
Update UI #135
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.
Using a parallel route results in a backend error if the user is not part of any chapter. I.e, their chapter ID will be undefined.
(user) => user.role === "CHAPTER_LEADER" | ||
); | ||
|
||
// const options: Parameters<typeof TileEdit>[0]["options"] = []; |
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.
nit: remove commented-out code if not needed
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
Description
Update the user's tile to join chapters to match the new Figma styling.
Screenshots
By default, new users can join any chapters in the Legacy Project.
Having 1 pending chapter request will block the user from joining any other chapters.
Additional Documentations
I also fixed a bug with parallel routing. See comments for more information.