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

add vss config option #559

Merged
merged 7 commits into from
Sep 29, 2023
Merged

add vss config option #559

merged 7 commits into from
Sep 29, 2023

Conversation

futurepaul
Copy link
Collaborator

used this opportunity to improve how we handle setup errors outside the main route. before it would hang if error when trying to load /settings/servers. still not a PERFECT story for if someone puts in a bad url and saves, but they should still be able to nav to settings and fix. I made it so settings doesn't even rely on megastore which should make it extra resilient. could potentially add a link on an error page but would only want to do that if we were confident that's the right move

@cloudflare-pages
Copy link

cloudflare-pages bot commented Sep 27, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 062ee5e
Status: ✅  Deploy successful!
Preview URL: https://e8402453.mutiny-web.pages.dev
Branch Preview URL: https://vss-configurable.mutiny-web.pages.dev

View logs

@benthecarman benthecarman force-pushed the vss-configurable branch 2 times, most recently from 9410434 to e17d7fa Compare September 28, 2023 01:33
@TonyGiorgio TonyGiorgio merged commit 8bd17db into master Sep 29, 2023
4 checks passed
@TonyGiorgio TonyGiorgio deleted the vss-configurable branch September 29, 2023 01:03
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.

None yet

3 participants