Skip to content
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(76): zustand removal & structure refactor #80

Merged
merged 29 commits into from
Mar 22, 2024

Conversation

RyRy79261
Copy link
Contributor

@RyRy79261 RyRy79261 commented Mar 14, 2024

Overview

This PR restructures the repo to align with NextJS app folder structure as well as remove zustand based store management.

Reviewing

Given the wide scope of changes, and the development processes going forward, I would recommend having a group walkthrough to review

Changelog

  • zustand state removed and moved into Tanstack via wagmi
  • GH Action for linting updated to use PNPM
  • Docs updated to use pnpm
  • Eslint updated to use typescript recommended
  • Folder structure refactored to align with nextjs app folder recommendations

Related issues

@RyRy79261 RyRy79261 self-assigned this Mar 14, 2024
Copy link

vercel bot commented Mar 14, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
governance-ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 22, 2024 0:14am

@chapati23
Copy link
Contributor

chapati23 commented Mar 18, 2024 via email

@denviljclarke
Copy link
Contributor

Nice work @RyRy79261, it looks good overall. I left a few comment & scheduled a walkthrough with you 😁

AnnaJ-P11 and others added 3 commits March 21, 2024 12:28
Co-authored-by: denviljclarke <60730266+denviljclarke@users.noreply.github.com>
@RyRy79261 RyRy79261 merged commit c1c92ad into develop Mar 22, 2024
3 checks passed
@RyRy79261 RyRy79261 deleted the feat/76-zustand-removal branch March 22, 2024 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants