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

[omdb] show sled policy and state, allow application of filter #5620

Conversation

sunshowers
Copy link
Contributor

@sunshowers sunshowers commented Apr 25, 2024

I realized that we have this wonderful SledFilter enum lying around, and we
can just use it in omdb.

I decided to shorten "eligible-for-discretionary-services" to "discretionary",
which I hope communicates the same meaning but in a shorter manner.

Depends on #5615.

Created using spr 1.3.6-beta.1

[skip ci]
Created using spr 1.3.6-beta.1
Copy link
Contributor

@jgallagher jgallagher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

jgallagher and others added 2 commits April 25, 2024 12:28
Created using spr 1.3.6-beta.1

[skip ci]
Created using spr 1.3.6-beta.1
@sunshowers sunshowers changed the base branch from sunshowers/spr/main.omdb-show-sled-policy-and-state-allow-application-of-filter to main April 25, 2024 19:28
@sunshowers sunshowers enabled auto-merge (squash) April 25, 2024 19:28
@sunshowers sunshowers merged commit 05839a4 into main Apr 26, 2024
26 checks passed
@sunshowers sunshowers deleted the sunshowers/spr/omdb-show-sled-policy-and-state-allow-application-of-filter branch April 26, 2024 01:01
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.

2 participants