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

Doc: Update wording for local run and 'Google Cloud Console' name #2329

Closed
wants to merge 4 commits into from
Closed

Conversation

steren
Copy link
Contributor

@steren steren commented Jan 3, 2017

* The official App Engine docs do not use the term "deploy" when running the app locally (https://cloud.google.com/appengine/docs/java/tools/devserver). It's better to only use the word "deploy" when deploying to App Engine.

* The Developer Console where users can setup App Engine is the Google Cloud Console (https://console.cloud.google.com)
@bazel-io
Copy link
Member

bazel-io commented Jan 3, 2017

Can one of the admins verify this patch?

@steren
Copy link
Contributor Author

steren commented Jan 3, 2017

Actually, let me add another change, because I realize that now App Engine app are not created automatically on project creation.
We need one extra click after project creation.

@steren
Copy link
Contributor Author

steren commented Jan 3, 2017

Done, I used the same link as the one used on the official doc: https://cloud.google.com/appengine/docs/java/quickstart#deploy_your_app

@bazel-io bazel-io closed this in e86afd9 Jan 3, 2017
steren added a commit to bazelbuild/bazel-website that referenced this pull request Apr 26, 2017
* The official App Engine docs do not use the term "deploy" when running the app locally (https://cloud.google.com/appengine/docs/java/tools/devserver). It's better to only use the word "deploy" when deploying to App Engine.

* The Console where users can setup App Engine is the Google Cloud Console (https://console.cloud.google.com), not the Developer Console

Closes #2329.

--
Reviewed-on: bazelbuild/bazel#2329
PiperOrigin-RevId: 143460661
MOS_MIGRATED_REVID=143460661
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants