-
Notifications
You must be signed in to change notification settings - Fork 51
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
Side panel #525
Side panel #525
Conversation
@pavloMandryk is attempting to deploy a commit to the 1hive Team on Vercel. A member of the Team first needs to authorize it. |
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.
Awesome man! just some small comments
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.
@pavloMandryk great work. Super smooth. I left several cosmetic changes to the name descriptions.
One thing that I am not convinced is about the CircleGraph
component showing both the %
of the progress and also the 1/6
step we are. Wonder if we shoudl keep the steps there 🤔
I have made some changes based on your suggestions:
|
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.
Well done! Looking great. Let's merge it.
This PR contains the reorganization of the steps, so we can have the same order as shows on issue #217 , and the implementation of substeps.