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 support for additional hostnames #28

Merged
merged 2 commits into from
Nov 21, 2022
Merged

add support for additional hostnames #28

merged 2 commits into from
Nov 21, 2022

Conversation

tyler36
Copy link
Collaborator

@tyler36 tyler36 commented Nov 11, 2022

Addreses #27

I'm not sure how many people are using additional hostnames. I assume people would prefer to see their project url instead of an IP in their terminal.

@tyler36
Copy link
Collaborator Author

tyler36 commented Nov 11, 2022

This breaks auto-updates by asking people to remove "ddev-generated".. I'm not sure if this an ideal tradeoff.

@tyler36
Copy link
Collaborator Author

tyler36 commented Nov 14, 2022

Refactored to support additional host out of the box

@tyler36 tyler36 merged commit f9331e4 into main Nov 21, 2022
@tyler36 tyler36 deleted the tyler36/issue27 branch November 21, 2022 23:56
@rfay
Copy link
Member

rfay commented Nov 22, 2022

Congrats!

@tyler36
Copy link
Collaborator Author

tyler36 commented Nov 22, 2022

I'm actually restesting it now ... and it doesn't appear to be working. I might need to revert it.

@tyler36 tyler36 restored the tyler36/issue27 branch November 22, 2022 00:11
tyler36 added a commit that referenced this pull request Nov 22, 2022
tyler36 added a commit that referenced this pull request Nov 22, 2022
Revert "add support for additional hostnames (#28)"

This reverts commit f9331e4.
@tyler36 tyler36 deleted the tyler36/issue27 branch February 3, 2023 03:35
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.

2 participants