-
Notifications
You must be signed in to change notification settings - Fork 11
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: members list #20
Conversation
@pbkompasz is attempting to deploy a commit to the BuidlGuidl Team on Vercel. A member of the Team first needs to authorize it. |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@phipsae I added the builder profiles that are currently merged, the rest will have to be added afterwards. |
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.
@pbkompasz, thank you for taking on the issue!
There are still a few things that need attention, particularly implementing an automated approach for retrieving the builder profiles.
Please see my comments below.
If there are any questions, just ask me! Thank you!
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.
Thank you! It’s working well. However, I’d avoid using a constant for the names, as we’d have to manually update the codebase each time a new builder is added. Also, let’s ensure no errors are thrown in the console.
Thanks a lot for your work! It works great, let's merge and we finally have a nice overview page for all the builders. |
Description
Adds a simple list containing the checked in members' builder page.
Additional Information
Related Issues
Closes #4
Your ENS/address: 0x26BfbD8ED2B302ec2c2B6f063C4caF7abcB062e0