Skip to content

ui impl for labels include all#41836

Merged
jahzielv merged 4 commits intofeat-installers-labels-include-allfrom
40724-include-all-fe
Mar 18, 2026
Merged

ui impl for labels include all#41836
jahzielv merged 4 commits intofeat-installers-labels-include-allfrom
40724-include-all-fe

Conversation

@jahzielv
Copy link
Copy Markdown
Contributor

@jahzielv jahzielv commented Mar 17, 2026

Related issue: Resolves #40724

Checklist for submitter

If some of the following don't apply, delete the relevant line.

Testing

  • Added/updated automated tests
  • QA'd all new/changed functionality manually

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 18, 2026

Codecov Report

❌ Patch coverage is 12.12121% with 29 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (feat-installers-labels-include-all@02a9eb8). Learn more about missing BASE report.

Files with missing lines Patch % Lines
frontend/services/entities/software.ts 5.55% 17 Missing ⚠️
frontend/pages/SoftwarePage/helpers.tsx 20.00% 12 Missing ⚠️
Additional details and impacted files
@@                          Coverage Diff                          @@
##             feat-installers-labels-include-all   #41836   +/-   ##
=====================================================================
  Coverage                                      ?   66.31%           
=====================================================================
  Files                                         ?     2501           
  Lines                                         ?   198693           
  Branches                                      ?     8920           
=====================================================================
  Hits                                          ?   131754           
  Misses                                        ?    55042           
  Partials                                      ?    11897           
Flag Coverage Δ
frontend 54.38% <12.12%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@jahzielv jahzielv marked this pull request as ready for review March 18, 2026 18:58
@jahzielv jahzielv requested a review from a team as a code owner March 18, 2026 18:58
@jahzielv jahzielv linked an issue Mar 18, 2026 that may be closed by this pull request
@jahzielv jahzielv merged commit 8d646cd into feat-installers-labels-include-all Mar 18, 2026
17 checks passed
@jahzielv jahzielv deleted the 40724-include-all-fe branch March 18, 2026 19:16
@jahzielv jahzielv mentioned this pull request Mar 19, 2026
10 tasks
georgekarrv added a commit that referenced this pull request Mar 19, 2026
- **DB migration: Add `require_all` column to installers labels tables
(#41279)**
- **Backend: Support labels_include_all for installers/apps (#41324)**
- **ui impl for labels include all (#41836)**

<!-- Add the related story/sub-task/bug number, like Resolves #123, or
remove if NA -->
**Related issue:** Resolves #39916

# Checklist for submitter

If some of the following don't apply, delete the relevant line.

- [x] Changes file added for user-visible changes in `changes/`,
`orbit/changes/` or `ee/fleetd-chrome/changes`.
See [Changes
files](https://github.com/fleetdm/fleet/blob/main/docs/Contributing/guides/committing-changes.md#changes-files)
for more information.

- [x] Input data is properly validated, `SELECT *` is avoided, SQL
injection is prevented (using placeholders for values in statements), JS
inline code is prevented especially for url redirects, and untrusted
data interpolated into shell scripts/commands is validated against shell
metacharacters.

## Testing

- [x] Added/updated automated tests
- [x] Where appropriate, [automated tests simulate multiple hosts and
test for host
isolation](https://github.com/fleetdm/fleet/blob/main/docs/Contributing/reference/patterns-backend.md#unit-testing)
(updates to one hosts's records do not affect another)

- [x] QA'd all new/changed functionality manually

## Database migrations

- [x] Checked schema for all modified table for columns that will
auto-update timestamps during migration.

## New Fleet configuration settings

- [x] Verified that the setting is exported via `fleetctl
generate-gitops`
- [x] Verified the setting is documented in a separate PR to [the GitOps
documentation](https://github.com/fleetdm/fleet/blob/main/docs/Configuration/yaml-files.md#L485)
- [x] Verified that the setting is cleared on the server if it is not
supplied in a YAML file (or that it is documented as being optional)
- [x] Verified that any relevant UI is disabled when GitOps mode is
enabled
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.

FE: UI dropdown, API integration

2 participants