Update cli monorepo ethers and subgraph web app UI

[yagopajarino]

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

• My code follows the style guidelines of this project
• I have performed a self-review of my code
• I have commented my code, particularly in hard-to-understand areas
• My changes generate no new warnings
• I have run yarn format and yarn lint without getting any errors
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes

[vplasencia]

Hey @yagopajarino!

If you run yarn in the project root directory, do you see that the yarn.lock file was modified? If so, can you also push these changes?

[yagopajarino]

Pushed the new yarn.lock :)

[cedoor]

Hi @yagopajarino, could you solve the conflicts?

[yagopajarino]

Hi @cedoor! I just synced the fork and solved the conflicts, let me know if there are any additional changes needed

[cedoor]

@yagopajarino looks like there're still conflicts.

[yagopajarino]

Hi @cedoor! Fixed the conflicts

[yagopajarino]

Hi @vplasencia made some changes, let me know what do you think :)

Btw thanks a lot for the reviews

[vplasencia]

Hey! I just left a few comments to improve the UI.

Also, the icons and styles folders inside the cli-template-monorepo-subgraph can be removed since those are used in a chakra project.

I think that after these small updates, the PR is ready to be merged. Thank you very much for the great work! 🙏

[yagopajarino]

Thanks for the feedback! I made the changes

[vplasencia]

Hey! Thank you for the updates.

We can remove the LogContext.ts and SemaphoreContext.ts from both templates and just keep the new ones with tsx extension.

I just noticed that in the Semaphore Subgraph template, we should use import { SemaphoreSubgraph } from "@semaphore-protocol/data" instead of import { SemaphoreEthers } from "@semaphore-protocol/data". The folder structure won't change.

For more reference:

• Current Subgraph template

• docs

[yagopajarino]

Hi @vplasencia! Made the changes

[vplasencia]

Hey @yagopajarino thank you. Could you also remove the useSemaphore.ts file inside the hooks folder in both web apps?

[yagopajarino]

Done @vplasencia 🚀

[vplasencia]

Thank you very much for the great work @yagopajarino! 🙏

[gitpoap-bot]

Congrats, your important contribution to this open-source project has earned you a GitPOAP!

GitPOAP: 2024 Semaphore Contributor:

Head to gitpoap.io & connect your GitHub account to mint!

Learn more about GitPOAPs here.