Skip to content

feat: responsive design, deployment config, and lint fixes#40

Merged
Iwueseiter merged 1 commit into
HubDApp:mainfrom
mayasimi:feature/project-discovery-filter-sort
Apr 28, 2026
Merged

feat: responsive design, deployment config, and lint fixes#40
Iwueseiter merged 1 commit into
HubDApp:mainfrom
mayasimi:feature/project-discovery-filter-sort

Conversation

@mayasimi
Copy link
Copy Markdown
Contributor

close #29
Responsiveness:

  • Remove double LayoutWrapper from all pages (root layout already wraps)
  • Fix profile/page.tsx and submit/page.tsx missing layout structure
  • Add hide-scrollbar utility to globals.css (used in discover page)
  • Sync Navbar mobile menu to include all 5 nav links (Reviews, Verify)
  • Fix all pages to use consistent pt-8 pb-24 spacing

Deployment:

  • Configure next.config.ts: reactStrictMode, compress, image domains, security headers (X-Frame-Options, CSP, Referrer-Policy, etc.)
  • Add ProjectRegistrationParams interface to soroban.service.ts

Lint fixes (0 errors, 0 warnings):

  • admin/page.tsx: replace useEffect+setState with derived isAdmin value
  • reviews/page.tsx: lazy useState initializer instead of useEffect
  • discover/page.tsx: replace useEffect pagination reset with page state and filter-aware setter helpers
  • VerificationStatus.tsx: IIFE pattern for async effect, lazy init state
  • useStellarAccount.ts: IIFE pattern for async effect
  • wallet.context.tsx: move disconnectWallet before polling effect, add to dependency array
  • ReviewForm.tsx: remove unused prop destructuring
  • ProjectForm.tsx: remove unused projectId from destructuring

Build: all 10 routes compile (9 static, 1 dynamic)
Tests: 27/27 passing

Responsiveness:
- Remove double LayoutWrapper from all pages (root layout already wraps)
- Fix profile/page.tsx and submit/page.tsx missing layout structure
- Add hide-scrollbar utility to globals.css (used in discover page)
- Sync Navbar mobile menu to include all 5 nav links (Reviews, Verify)
- Fix all pages to use consistent pt-8 pb-24 spacing

Deployment:
- Configure next.config.ts: reactStrictMode, compress, image domains,
  security headers (X-Frame-Options, CSP, Referrer-Policy, etc.)
- Add ProjectRegistrationParams interface to soroban.service.ts

Lint fixes (0 errors, 0 warnings):
- admin/page.tsx: replace useEffect+setState with derived isAdmin value
- reviews/page.tsx: lazy useState initializer instead of useEffect
- discover/page.tsx: replace useEffect pagination reset with page state
  and filter-aware setter helpers
- VerificationStatus.tsx: IIFE pattern for async effect, lazy init state
- useStellarAccount.ts: IIFE pattern for async effect
- wallet.context.tsx: move disconnectWallet before polling effect,
  add to dependency array
- ReviewForm.tsx: remove unused prop destructuring
- ProjectForm.tsx: remove unused projectId from destructuring

Build: all 10 routes compile (9 static, 1 dynamic)
Tests: 27/27 passing
@drips-wave
Copy link
Copy Markdown

drips-wave Bot commented Apr 27, 2026

@mayasimi Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Iwueseiter Iwueseiter merged commit e1082d6 into HubDApp:main Apr 28, 2026
1 check passed
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.

Ensure app responsive and deployment ready

2 participants