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: Redesign start page #30

Merged
merged 11 commits into from
Jan 8, 2024
Merged

feat: Redesign start page #30

merged 11 commits into from
Jan 8, 2024

Conversation

zoey-kaiser
Copy link
Member

@zoey-kaiser zoey-kaiser commented Oct 12, 2023

Pre-work for next major release.

  • Redesigns start page
  • Moves Modal examples from pages to components and integrates them into one page
  • Improves design and matches it more with our other branding (like docs page)

Screenshot 2023-10-12 at 16 51 10

Screen.Recording.2023-10-12.at.16.51.47.mov

@@ -48,4 +48,4 @@ jobs:
- run: "export AUTH_ORIGIN=http://localhost:3000 && export AUTH_SECRET=test123 && cd my-sidebase-app && npm run build && timeout 30 npm run preview || ( [[ $? -eq 124 ]] && echo \"app started and did not exit within first 30 seconds, thats good\" )"

# start dev-app and curl from it
- run: "cd my-sidebase-app && timeout 30 npm run dev & (sleep 10 && curl --fail localhost:3000) || ( [[ $? -eq 124 ]] && echo \"app started and did not exit within first 30 seconds, thats good\" )"
- run: "cd my-sidebase-app && timeout 30 npm run dev & (sleep 20 && curl --fail localhost:3000) || ( [[ $? -eq 124 ]] && echo \"app started and did not exit within first 30 seconds, thats good\" )"
Copy link
Member Author

@zoey-kaiser zoey-kaiser Oct 12, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Due to moving all the demos to the start page, I had to increase the sleep timer here to ensure all the data could load, before the curl was performed.

Note: This did happen sometimes before, where the curl was run before the page had finished starting up. See https://github.com/sidebase/create-sidebase/actions/runs/6500004358/job/17654497392?pr=30 where we pushed #29 to main.

@zoey-kaiser zoey-kaiser merged commit 24d0c65 into main Jan 8, 2024
2 checks passed
@zoey-kaiser zoey-kaiser deleted the update-design branch January 8, 2024 17:54
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.

1 participant