-
Notifications
You must be signed in to change notification settings - Fork 208
Update domain messages and text to be more domain centric (rather than namespace) #454
Update domain messages and text to be more domain centric (rather than namespace) #454
Conversation
@@ -535,7 +535,7 @@ def ask_for_namespace | |||
namespace = nil | |||
paragraph do | |||
begin | |||
namespace = ask "Please enter a namespace (letters and numbers only)#{namespace_optional? ? " |<none>|" : ""}: " do |q| | |||
namespace = ask "Please enter a domain name (letters and numbers only)#{namespace_optional? ? " |<none>|" : ""}: " do |q| |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the only place where I think people might try to enter 'foo.com'
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changing it back
reviewed |
reviewed, [merge] |
Online Merge Results: SUCCESS (https://ci.dev.openshift.redhat.com/jenkins/job/test_pull_requests/4194/) (Image: devenv_3755) |
Evaluated for online up to 9620d2c |
[Test]ing while waiting on the merge queue |
Online Test Results: SUCCESS (https://ci.dev.openshift.redhat.com/jenkins/job/test_pull_requests/4194/) |
Origin Test Results: Running (https://originci-openshift.rhcloud.com/job/test_pull_requests/1691/) |
Evaluated for origin up to 9620d2c |
@liggitt review