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 - add !pipeline filter #4016

Merged
merged 4 commits into from
Aug 31, 2023

Conversation

himdel
Copy link
Collaborator

@himdel himdel commented Jul 21, 2023

add filter, no need to use fixed repos to filter approved/rejected out now

don't expect to get the full list of repositories
=> only use loadAllRepos for repaint and preselect,
=> no select all
=> use selected counts for deselect

@github-actions github-actions bot added the backport-4.7 This PR should be backported to stable-4.7 (2.4) label Jul 21, 2023
@himdel himdel marked this pull request as ready for review August 31, 2023 18:02
add filter, no need to use fixed repos to filter approved/rejected out now

also switch to AnsibleRepositoryAPI - no need to rewrite sort & ordering

don't expect to get the full list of repositories
=> only use loadAllRepos for repaint and preselect,
=> no select all
=> use selected counts for deselect

No-Issue
@himdel himdel merged commit 07a1c1c into ansible:master Aug 31, 2023
16 of 19 checks passed
@himdel himdel deleted the import-modal-pipeline branch August 31, 2023 23:44
@patchback
Copy link

patchback bot commented Aug 31, 2023

Backport to stable-4.7: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-4.7/07a1c1cd45e400969c1990df435d7523589ae2af/pr-4016

Backported as #4164

🤖 @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 Aug 31, 2023
* Import modal - add !pipeline filter

add filter, no need to use fixed repos to filter approved/rejected out now

also switch to AnsibleRepositoryAPI - no need to rewrite sort & ordering

don't expect to get the full list of repositories
=> only use loadAllRepos for repaint and preselect,
=> no select all
=> use selected counts for deselect

No-Issue

* repo selector table - use <td> for individual fields

* tests: select all is gone, deselect all shows actual selected count

* click N more before testing labels

(cherry picked from commit 07a1c1c)
himdel added a commit that referenced this pull request Sep 1, 2023
* Import modal - add !pipeline filter

add filter, no need to use fixed repos to filter approved/rejected out now

also switch to AnsibleRepositoryAPI - no need to rewrite sort & ordering

don't expect to get the full list of repositories
=> only use loadAllRepos for repaint and preselect,
=> no select all
=> use selected counts for deselect

No-Issue

* repo selector table - use <td> for individual fields

* tests: select all is gone, deselect all shows actual selected count

* click N more before testing labels

(cherry picked from commit 07a1c1c)

Co-authored-by: Martin Hradil <mhradil@redhat.com>
@github-actions github-actions bot added the backported-4.7 This PR has been backported to stable-4.7 (2.4) label Sep 1, 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) backported-4.7 This PR has been backported to stable-4.7 (2.4)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant