-
Notifications
You must be signed in to change notification settings - Fork 194
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 cli monorepo ethers and subgraph web app UI #841
base: main
Are you sure you want to change the base?
Conversation
…o-subgraph web app UI re semaphore-protocol#836
Hey @yagopajarino! If you run |
Pushed the new |
Hi @yagopajarino, could you solve the conflicts? |
Hi @cedoor! I just synced the fork and solved the conflicts, let me know if there are any additional changes needed |
@yagopajarino looks like there're still conflicts. |
Hi @cedoor! Fixed the conflicts |
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.
Hey @yagopajarino!
Thank you very much for the PR. Great work! 🙏
Just one comment, could you remove Chakra UI from the two apps and use css like in the current templates? The idea is to make it easier for developers to integrate the css library/framework they prefer, then using plain css is better for this case.
Hello @vplasencia! I changed the .env and the .gitignore of both templates. In upcoming days I'll be replacing the chakra elements for plain html + css :) |
Hey @yagopajarino! Thank you very much. No rush, take your time. |
Description
Update cli template ethers and subgraph web app UI using boilerplate code from https://github.com/semaphore-protocol/boilerplate/tree/main/apps/web-app
Related Issue(s)
Closes #836
Checklist
yarn format
andyarn lint
without getting any errors