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

[WIP] Feat: Support for adding Custom Domains with railway domain [domain] [-p | --port] #571

Merged
merged 9 commits into from
Nov 19, 2024

Conversation

alexng353
Copy link
Contributor

@alexng353 alexng353 commented Nov 4, 2024

Usage: railway domain [domain] [-p | --port]

Also can run interactively. Currently am unsure if it will work headless, but willing to test if that's necessary for this feature.

The command runs entirely headlessly. If you are creating a custom domain in headless mode, I would recommend using the --json flag to get a clean JSON output of the DNS records.

Note: I updated the GQL schema using the method provided in CONTRIBUTING.md, which can be reverted if the admin deems it necessary (for whatever reason).

Docs PR: railwayapp/docs#613

Fixes: #512

image

@alexng353 alexng353 force-pushed the alex/add-custom-domains branch from e5322ed to 7a58067 Compare November 4, 2024 09:05
@brody192 brody192 changed the title Feat: CLI command to add a custom domain [WIP] Feat: CLI command to add a custom domain Nov 4, 2024
@alexng353 alexng353 changed the title [WIP] Feat: CLI command to add a custom domain [WIP] Feat: Support for adding Custom Domains with railway domain [domain] Nov 5, 2024
src/commands/domain.rs Outdated Show resolved Hide resolved
@alexng353 alexng353 force-pushed the alex/add-custom-domains branch from af19202 to 6b3b14b Compare November 6, 2024 19:09
Copy link
Contributor

@coffee-cup coffee-cup left a comment

Choose a reason for hiding this comment

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

Looking good! Just a few comments

src/commands/domain.rs Outdated Show resolved Hide resolved
src/commands/domain.rs Outdated Show resolved Hide resolved
src/commands/domain.rs Show resolved Hide resolved
src/commands/domain.rs Show resolved Hide resolved
src/commands/domain.rs Show resolved Hide resolved
@coffee-cup coffee-cup added the release/minor Author minor release label Nov 18, 2024
@alexng353 alexng353 changed the title [WIP] Feat: Support for adding Custom Domains with railway domain [domain] [WIP] Feat: Support for adding Custom Domains with railway domain [domain] [-p | --port port] Nov 19, 2024
@alexng353 alexng353 changed the title [WIP] Feat: Support for adding Custom Domains with railway domain [domain] [-p | --port port] [WIP] Feat: Support for adding Custom Domains with railway domain [domain] [-p | --port] Nov 19, 2024
Copy link
Contributor

@coffee-cup coffee-cup left a comment

Choose a reason for hiding this comment

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

🎉

@coffee-cup coffee-cup merged commit a849e2b into railwayapp:master Nov 19, 2024
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release/minor Author minor release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add CLI feature for adding a custom domain
3 participants