chore: migration to NextJS + materialUI | upgrade wagmi + rainbowkit + viem #7

0xtiti · 2024-03-21T13:03:47Z

migration to NextJS + materialUI | upgrade wagmi + rainbowkit + viem

Checklist before requesting a review

I have conducted a self-review of my code.
If it is a core feature, I have included comprehensive tests.
Do we need to implement analytics?
Will this be part of a product update? If yes, please write one sentence about this update.

chore: wagmi, rainbowkit, viem updates tests: rainbokit 2.x breaking changes updates fix: injected wallet for empty projectId logic fix: wallets type error - projectid not required rainbowWallet nor walletConnect fix: caret removed fix: group imports chore: update viem and wagmi to latest version (#1) fix: yarn libs error

* chore: nextJS migration | *removed styles due to near future materialUI migration* * test: vitest adapted to nextJS environment, simple testing * fix: test type assertion | build type assertion fixes * fix: removed build info file * fix: added tsconfig.tsbuildinfo to gitignore - false possitive on private keys * fix: removed caret * fix: reinstalled yarn.lock build issues * fix: build issue with node version * fix: update nodejs version NextJS migration nodeJS version issue * fix: Ensure using correct Node.js version in GitHub Actions * fix: nodejs version issue * fix: revert back to nodejs -v 18.7 * fix: resolve "x" type + udpate version 21.4 * fix: remove carets * fix: remove carets * fix: footer/header tags | group imports | sort imports * fix: next.config default * fix: eslintrc remove typeprops rule + default file * fix: remove unused libs and old tests * fix: remove vite and vitest * fix:lint issue * fix: layout children proptype validation * fix: remove test - cypress migration * fix: added cypress and pnpm * fix: remove caret

* refactor: nextJS architecture improvement * fix: materialUI styling as per styled components version * fix: building materialUI Theme bug * fix: remove carets | imports sorting * fix: wallet connect issues + sepolia | mobile view alert * fix: remove use client * chore: misc fixes * chore: update pnpm-lock file --------- Co-authored-by: Ardy <[email protected]>

vercel · 2024-03-21T13:03:52Z

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

Name	Status	Preview	Updated (UTC)
web3-react-boilerplate	✅ Ready (Inspect)	Visit Preview	Mar 21, 2024 1:03pm

0xArdy · 2024-03-21T13:23:11Z

@turtlemoji what do you think about changing yarn to pnpm? We use pnpm in the superchain bridge repo bc yarn was giving problems when building and running the app locally with nextjs

turtlemoji · 2024-03-22T14:56:38Z

@turtlemoji what do you think about changing yarn to pnpm? We use pnpm in the superchain bridge repo bc yarn was giving problems when building and running the app locally with nextjs

I'm ok with pnpm, but I don't know If I like the idea to setup nextjs as default. Not all projects need it.

package.json

cypress/support/commands.ts

.env.example

turtlemoji · 2024-03-22T15:26:52Z

cypress/e2e/fundamentals.cy.ts

@@ -0,0 +1,5 @@
+describe('template spec', () => {
+  it('passes', () => {
+    cy.visit('https://example.cypress.io');


change default examples, maybe

turtlemoji · 2024-03-22T15:27:13Z

cypress/fixtures/example.json

@@ -0,0 +1,5 @@
+{


change or delete examples

package.json

src/config/env.ts

src/containers/Footer/index.tsx

src/containers/Header/index.tsx

src/containers/Landing/index.tsx

src/pages/layout.tsx

src/providers/Web3ModalProvider.tsx

src/pages/_document.tsx

turtlemoji · 2024-03-22T16:58:13Z

src/hooks/ScrollToTop.tsx

+    router.events.on('routeChangeComplete', handleRouteChange);
+
+    return () => {
+      router.events.off('routeChangeComplete', handleRouteChange);


recheck for memory leaks / this is working as expected

0xtiti and others added 3 commits March 15, 2024 10:44

0xtiti requested review from turtlemoji and 0xArdy March 21, 2024 13:14