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

Import modal - respect display_repositories, hide selector when no choices #4458

Merged
merged 2 commits into from
Oct 31, 2023

Conversation

himdel
Copy link
Collaborator

@himdel himdel commented Oct 31, 2023

When the display_repositories feature flag is disabled, don't show the "Staging Repos"/"All Repos" radios, only allow staging.

When there is only 1 candidate choice within the current list, don't show the whole list, only the preselected repo.

Thus...

display_repositories == false, staging count == 1:
20231031034619

display_repositories == true, staging count == 1:
20231031034653

display_repositories == false, staging count > 1:
20231031040227

display_repositories == true, staging count > 1:
20231031035136

@github-actions github-actions bot added backport-4.7 This PR should be backported to stable-4.7 (2.4) backport-4.8 This PR should be backported to stable-4.8 (2.4) labels Oct 31, 2023
@himdel himdel added the community comunity mode feature label Oct 31, 2023
@himdel himdel force-pushed the collection-upload branch from 0743e21 to 221acd9 Compare October 31, 2023 04:07
@gundalow
Copy link

display_repositories == false, staging count == 1: screenshot is a lot clearer, thank you.

@himdel himdel force-pushed the collection-upload branch from 221acd9 to f0b84e6 Compare October 31, 2023 19:28
@himdel himdel merged commit 9c8e101 into ansible:master Oct 31, 2023
@himdel himdel deleted the collection-upload branch October 31, 2023 20:37

This comment was marked as outdated.

Copy link

patchback bot commented Oct 31, 2023

Backport to stable-4.8: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-4.8/9c8e10119d8e9ba0d924424b1998892ca54a3656/pr-4458

Backported as #4464

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Oct 31, 2023
…oices (#4458)

* Import modal - respect display_repositories, hide selector when there are no choices

No-Issue

* tests

(cherry picked from commit 9c8e101)
himdel added a commit to himdel/ansible-hub-ui that referenced this pull request Oct 31, 2023
…oices (ansible#4458)

* Import modal - respect display_repositories, hide selector when there are no choices

No-Issue

* tests

(cherry picked from commit 9c8e101)
himdel added a commit that referenced this pull request Oct 31, 2023
…oices (#4458) (#4464)

* Import modal - respect display_repositories, hide selector when there are no choices

No-Issue

* tests

(cherry picked from commit 9c8e101)

Co-authored-by: Martin Hradil <mhradil@redhat.com>
@github-actions github-actions bot added the backported-4.8 This PR has been backported to stable-4.8 (2.4) label Oct 31, 2023
himdel added a commit that referenced this pull request Oct 31, 2023
…oices (#4458) (#4465)

* Import modal - respect display_repositories, hide selector when there are no choices

No-Issue

* tests

(cherry picked from commit 9c8e101)
@github-actions github-actions bot added the backported-4.7 This PR has been backported to stable-4.7 (2.4) label Oct 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-4.7 This PR should be backported to stable-4.7 (2.4) backport-4.8 This PR should be backported to stable-4.8 (2.4) backported-4.7 This PR has been backported to stable-4.7 (2.4) backported-4.8 This PR has been backported to stable-4.8 (2.4) community comunity mode feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants