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

Merge to main and deploy to Cloudflare #37

Merged
merged 67 commits into from
Feb 20, 2025
Merged

Merge to main and deploy to Cloudflare #37

merged 67 commits into from
Feb 20, 2025

Conversation

earth2travis
Copy link
Contributor

No description provided.

skuhlmann and others added 30 commits February 11, 2025 08:48
- Add missing key prop to Link component in DaoList to prevent React warnings and improve rendering performance
- Update wrangler.toml configuration
- Add 'use client' directive to DaoList component to prevent hydration errors
- Add empty [vars] section to declare environment variables in wrangler.toml
- Remove [env] section from wrangler.toml
- Allow variable values to be managed via Cloudflare Dashboard secrets
- Add SSR detection to skip provider when running on server
- Prevent context errors during build time when graphKey is empty
- Maintain original provider functionality in browser environment
- Keep type safety and existing provider interface
- Remove edge runtime directive from DAO page to resolve static generation issues
- Add 'use client' directive to ensure DAO hooks run client-side only
- Update DaoHooksProvider to handle SSR context gracefully

This resolves the build error 'DaoHooksContext must be used within a DaoHooksProvider' during static page generation.
- Keep 'use client' directive for root page
- Use dynamic import for DaoList without SSR flag

This resolves the 'DaoHooksContext must be used within a DaoHooksProvider' error during static generation.
- Add mounted state to DaoList to prevent SSR hooks
- Mark root page as client component to support DaoList context

This resolves the DaoHooksContext error during static generation.
- Add force-dynamic export to root page
- Keep client component directive
- Maintain DaoList client-side rendering

This prevents prerender errors with DaoHooksContext during build.
Renames imported dynamic function to avoid collision with Next.js page config export
fix: add key prop to Link component in DaoList
feat: redesign connect view with centered gate
earth2travis and others added 28 commits February 12, 2025 23:26
- Move wallet connection and chain name logic to page component
- Simplify DaoList to focus on displaying DAOs
- Improve styling and layout
refactor: move connection UI from DaoList to page component
adds active prop fetch hook and display in dao list card
- Add Avatar component for DAO images
- Style cards
- Add truncation for long DAO names (>25 chars)
- Add Avatar component for DAO images
- Style cards
- Add truncation for long DAO names (>25 chars)
- Integrate active proposals count from dev
Display 'Active Proposal' for single proposal and 'Active Proposals' for zero or multiple proposals
- Add loading check for proposals data
- Rename loading states (daoLoading, proposalsLoading)
- Calculate proposalCount before render
- Handle both loading states before component render
- Add font preloading for FetteUNZFraktur
- Add text-rendering optimization
- Standardize font-face declarations
- Reduce Largest Contentful Paint (LCP) time
Make proposal and no DAOs view
…o "Make Proposal" across all forms\n- Standardize form label styles using ProposalFormLabel\n- Fix Description placeholder text capitalization\n- Normalize validation messages across all form fields\n- Update validation message styling to use destructive color\n- Clean up form field spacing and layout\n- Remove unused SampleForm component
Tweaking hard on proposals
- Hide form fields after confirmation
- Add heart SVG image to success state
Copy link

netlify bot commented Feb 20, 2025

Deploy Preview for celadon-stardust-69bc7a ready!

Name Link
🔨 Latest commit 363a816
🔍 Latest deploy log https://app.netlify.com/sites/celadon-stardust-69bc7a/deploys/67b6739e0a7f200008b95af8
😎 Deploy Preview https://deploy-preview-37--celadon-stardust-69bc7a.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@earth2travis earth2travis merged commit 990c957 into main Feb 20, 2025
6 checks 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.

2 participants