katello icon indicating copy to clipboard operation
katello copied to clipboard

Fixes #35565 - ACS UX fixes - Wizard

Open sjha4 opened this issue 2 years ago • 5 comments

What are the changes introduced in this pull request?

Several changes based on UX review for ACS UI

Considerations taken when implementing this change?

What are the testing steps for this pull request?

Note the changes highlighted with yellow. Those have been addressed with this PR. Test and make sure everything behaves as per review doc in issue.

sjha4 avatar Sep 26 '22 20:09 sjha4

Issues: #35565

theforeman-bot avatar Sep 26 '22 20:09 theforeman-bot

I want to see new ACS without being forced to refresh manually. Sometimes it works sometimes it doesn’t

x It doesn't (for me.) The page doesn't refresh on create after I finish the wizard..

Sorry, moved it to part 2. Ignore this for the scope of this PR.

sjha4 avatar Sep 28 '22 03:09 sjha4

I was able to enter http://ee; is this valid?

I didn't want to make too restrictive of validations around this. For ex: http://ee is valid if you update your /etc/hosts to point it to an IP.. There's also IP addresses etc which might be used. The only check I have here is that it contains a protocol:// 🐱

sjha4 avatar Sep 28 '22 03:09 sjha4

[test katello]

sjha4 avatar Sep 28 '22 13:09 sjha4

Edit alternate content source URL and subpaths

Does it make sense to drop "alternate content source" altogether and just have Edit URL and subpaths

sjha4 avatar Sep 28 '22 19:09 sjha4