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 better hostname support #106

Merged
merged 1 commit into from
Jul 16, 2022
Merged

Add better hostname support #106

merged 1 commit into from
Jul 16, 2022

Conversation

jpbogle
Copy link
Member

@jpbogle jpbogle commented Jul 16, 2022

This PR includes updated README for config options to adding a pool, and also specific instructions for adding a custom hostname

This uses some nextJS rewrite magic. Unfortunately it has 2 shortcomings which is the domain HAS to add your pools name on the path, it will not allow for rewriting / so there is both a rewrite and redirect rule added. In addition nextJS support for importing a ESM module into next.config.js does not work so right now you have to take 1 extra step to define the hostname mapping. Eventually we will figure out a way to have all pool config in one place

@vercel
Copy link

vercel bot commented Jul 16, 2022

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

Name Status Preview Updated
cardinal-staking-ui ✅ Ready (Inspect) Visit Preview Jul 16, 2022 at 9:26PM (UTC)

@jpbogle jpbogle merged commit a9a139b into main Jul 16, 2022
@jpbogle jpbogle deleted the hostname-support branch July 16, 2022 21:29
kevy0915 added a commit to kevy0915/staking-ui that referenced this pull request Sep 23, 2024
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