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

fix(google) : FR: Help text on GCE load balancer type selection screen #7044

Merged
merged 12 commits into from
Jun 7, 2019

Conversation

meerajose
Copy link
Contributor

Added the help text on each GCE Loadbalancer type so that can easily understand. This fix is for the below issue.
spinnaker/spinnaker#3537

@spinnakerbot
Copy link
Contributor

The following commits need their title changed:

Please format your commit title into the form:

<type>(<scope>): <subject>, e.g. fix(kubernetes): address NPE in status check

This allows us to easily generate changelogs & determine semantic version numbers when cutting releases. You can read more about commit conventions here.

@maggieneterval maggieneterval self-requested a review May 22, 2019 15:17
Copy link
Contributor

@maggieneterval maggieneterval left a comment

Choose a reason for hiding this comment

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

Thanks so much for adding this fix!

Please change the commit title prefix to fix(google): ... to prevent spinbot from complaining about it.

I am also adding our technical writer as a reviewer to look over the wording of the help text.

app/scripts/modules/google/src/help/gce.help.ts Outdated Show resolved Hide resolved
@maggieneterval
Copy link
Contributor

Also Dave just FYI here's a screenshot of how this change looks:

yeN1Jrtx3st

meerajose and others added 2 commits May 23, 2019 16:00
Co-Authored-By: Maggie Neterval <mneterval@google.com>
spinnaker/spinnaker#3537
Made the changes to show helptext in each of the Load Balancer Type
@meerajose meerajose changed the title fix(deck) : FR: Help text on GCE load balancer type selection screen fix(google) : FR: Help text on GCE load balancer type selection screen May 23, 2019
…spinnaker#3537

spinnaker/spinnaker#3537

Update app/scripts/modules/google/src/help/gce.help.ts

Co-Authored-By: Maggie Neterval <mneterval@google.com>

fix(google): FR: Help text on GCE load balancer type selection screen

spinnaker/spinnaker#3537
Made the changes to show helptext in each of the Load Balancer Type
Copy link
Contributor

@dorbin dorbin left a comment

Choose a reason for hiding this comment

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

app/scripts/modules/google/src/help/gce.help.ts Outdated Show resolved Hide resolved
Copy link
Contributor

@jtk54 jtk54 left a comment

Choose a reason for hiding this comment

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

LGTM.

@maggieneterval maggieneterval merged commit 94dc56d into spinnaker:master Jun 7, 2019
christopherthielen added a commit to christopherthielen/deck that referenced this pull request Jul 3, 2019
cdd6f23 chore(package): Just Update Prettier™
5cc4146 perf(google): avoid unnecessary fetching and filtering of gce images (spinnaker#7115)
92492c4 fix(google): allow SpEL in stack/detail in deploy stage (spinnaker#7105)
94dc56d fix(google): Help text on GCE load balancer type selection screen (spinnaker#7044)
christopherthielen added a commit that referenced this pull request Jul 3, 2019
cdd6f23 chore(package): Just Update Prettier™
5cc4146 perf(google): avoid unnecessary fetching and filtering of gce images (#7115)
92492c4 fix(google): allow SpEL in stack/detail in deploy stage (#7105)
94dc56d fix(google): Help text on GCE load balancer type selection screen (#7044)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants