8000 Docs: Update ephemeral subdomains, remove Oasis link by ocdtrekkie · Pull Request #3719 · sandstorm-io/sandstorm · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Docs: Update ephemeral subdomains, remove Oasis link #3719

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

Merged
merged 1 commit into from
Nov 24, 2024

Conversation

ocdtrekkie
Copy link
Collaborator
@ocdtrekkie ocdtrekkie commented Nov 15, 2024

Generally most users will see a ui- prefix in their ephemeral subdomains, so I added this to some here to make this obvious. Note that if someone has a wildcard host config like alpha-*.example.com, they might have a ephemeral domain that looks like alpha-ui-45sda64dsfsd464ggdsg5.example.com, but that is uncommon enough I am not worried about clarifying it here.

Also, people cannot try this on Oasis.

@ocdtrekkie ocdtrekkie added documentation Documentation issues ready-for-review We think this is ready for review labels Nov 15, 2024
@ocdtrekkie
Copy link
Collaborator Author

@orblivion Can you review this PR?

@orblivion
Copy link
Contributor

Change looks fine, but why "most" users? Why not all or none? Was this documentation wrong before this fix?

@orblivion
Copy link
Contributor

Is the wildcard hosting the only reason you say "most"? Then that answers my question. (in which case I'd say the documentation was wrong)

@orblivion
Copy link
Contributor

👍 to merge though

@ocdtrekkie
Copy link
Collaborator Author

Yeah, if you tamper with the wildcard config you may have more things around the URL but there should always be a ui- in it for a grain view as of several years ago, and I just don't think this page was updated to reflect that.

@ocdtrekkie ocdtrekkie merged commit a2094c7 into master Nov 24, 2024
2 of 4 checks passed
@ocdtrekkie ocdtrekkie deleted the docs-ephemeral-urls branch November 24, 2024 04:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Documentation issues ready-for-review We think this is ready for review
Projects
None yet
Development

Successfully merging this pull request may 3811 close these issues.

2 participants
0