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

Updates as per UX team #189

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

elianerpereira
Copy link
Contributor

Kelsean from UX team requested some changed on wording and button labels.

Kelsean from UX team requested some changed on wording and button
labels.
@app-sre-bot
Copy link
Collaborator

Can one of the admins verify this patch?

@elianerpereira
Copy link
Contributor Author

I shared the link with Kelsea to get her review before asking for this PR to be merged.

@kel222ea
Copy link

Hey, @elianerpereira minor thing- now that I'm reading these over I think we should put the word "wizard" back next to "Image Builder". People may be confused and think they're already in Image Builder, but everyone in user interviews seems to know 'the wizard'.

@kel222ea
Copy link

@elianerpereira for the sentence I'm talking specifically line 26, just adding "wizard" after "Image Builder"

@Hyperkid123
Copy link
Collaborator

/ok-to-test

Hyperkid123
Hyperkid123 previously approved these changes May 14, 2024
@Hyperkid123 Hyperkid123 dismissed their stale review May 14, 2024 07:01

Missed review comment

@elianerpereira
Copy link
Contributor Author

@elianerpereira for the sentence I'm talking specifically line 26, just adding "wizard" after "Image Builder"

Thank you so much for the suggestion, @kel222ea , I added the word to avoid users to be confused.

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.

5 participants