Skip to content

[16533][fleet] fleetux you can enter an invalid name for a repository and continue#17044

Open
marcelofukumoto wants to merge 3 commits intorancher:masterfrom
marcelofukumoto:16533-fleetux-you-can-enter-an-invalid-name-for-a-repository-and-continue
Open

[16533][fleet] fleetux you can enter an invalid name for a repository and continue#17044
marcelofukumoto wants to merge 3 commits intorancher:masterfrom
marcelofukumoto:16533-fleetux-you-can-enter-an-invalid-name-for-a-repository-and-continue

Conversation

@marcelofukumoto
Copy link
Copy Markdown
Contributor

@marcelofukumoto marcelofukumoto commented Mar 27, 2026 •
edited
Loading

Summary

Fixes #16533

Occurred changes and/or fixed issues

  • Moved Name outside the wizard
  • Moved constraint from first step to last one.

Technical notes summary

  • Added UNIT TEST SNAPSHOTS for the TABS

Areas or cases that should be tested

  • AppBundles add repository
  • Edit AppBundles

Areas which could experience regressions

  • View AppBundles

Screenshot/Video

image image image image

Checklist

  • The PR is linked to an issue and the linked issue has a Milestone, or no issue is needed
  • The PR has a Milestone
  • The PR template has been filled out
  • The PR has been self reviewed
  • The PR has a reviewer assigned
  • The PR has automated tests or clear instructions for manual tests and the linked issue has appropriate QA labels, or tests are not needed
  • The PR has reviewed with UX and tested in light and dark mode, or there are no UX changes
  • The PR has been reviewed in terms of Accessibility
  • The PR has considered, and if applicable tested with, the three Global Roles Admin, Standard User and User Base

@marcelofukumoto
[16533] Added config to have NameNsDescription out of the flow
9ef0a0a 
Create a new slot to be top of the header.
Fixed the readines status
Fixed CruResource to have a reactive disabled state
@marcelofukumoto
[16533] Added tests for the cases and tests for the Tabs
1846551
@marcelofukumoto
[16533] Fixed UI to reduce margin
cf16e5b
@marcelofukumoto marcelofukumoto added this to the v2.15.0 milestone Mar 27, 2026
@marcelofukumoto marcelofukumoto marked this pull request as ready for review March 27, 2026 14:02
@edenhernandez-suse
Copy link
Copy Markdown
Contributor

edenhernandez-suse commented Apr 1, 2026

@kwwii we need to discuss this one.

@torchiaf
Copy link
Copy Markdown
Member

torchiaf commented Apr 9, 2026

The changes look good to me, but let's wait for the UX approval.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@torchiaf torchiaf Awaiting requested review from torchiaf

@edenhernandez-suse edenhernandez-suse Awaiting requested review from edenhernandez-suse

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

v2.15.0

Development

Successfully merging this pull request may close these issues.

[Fleet][UX] You can enter an invalid name for a Repository and continue

3 participants

@marcelofukumoto @edenhernandez-suse @torchiaf