Skip to content

Comments

Updates as per UX team#189

Open
elianerpereira wants to merge 2 commits intoRedHatInsights:mainfrom
elianerpereira:blueprint-fixes
Open

Updates as per UX team#189
elianerpereira wants to merge 2 commits intoRedHatInsights:mainfrom
elianerpereira:blueprint-fixes

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