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

Close #79 - Filtering on date fields within Hardware Lifecycle view #80

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

copacomilan
Copy link

@copacomilan copacomilan commented Feb 6, 2025

Closes: #79 - Add filtering on date fields for Hardware Lifecycle view

This commit adds the code for filtering hardware lifecycles on dates before user-defined end of sale, end of maintenance, end of security and/or end of support dates

This commit adds the code for filtering hardware lifecycles on dates before user-defined end of sale, end of maintenance, end of security and/or end of support dates
@copacomilan copacomilan changed the title end of sale/maintenance/security/support filters Close #79 - end of sale/maintenance/security/support filters Feb 6, 2025
@copacomilan copacomilan changed the title Close #79 - end of sale/maintenance/security/support filters Close #79 - Filtering on date fields within Hardware Lifecycle tab Feb 6, 2025
@copacomilan copacomilan changed the title Close #79 - Filtering on date fields within Hardware Lifecycle tab Close #79 - Filtering on date fields within Hardware Lifecycle view Feb 6, 2025
@DanSheps DanSheps enabled auto-merge February 6, 2025 21:10
@uck9
Copy link
Contributor

uck9 commented Feb 8, 2025

Quick check here - the forms\filtersets.py change is showing the same 4 datefield definitions (end_of_sale__lt). Was this meant to be the four different Fieldsets configured earlier?

For every DateField filter in forms/filtersets.py end_of_sale__lt was used as I copied and pasted it over from a local install, this is now fixed
auto-merge was automatically disabled February 10, 2025 12:38

Head branch was pushed to by a user without write access

@copacomilan
Copy link
Author

Quick check here - the forms\filtersets.py change is showing the same 4 datefield definitions (end_of_sale__lt). Was this meant to be the four different Fieldsets configured earlier?

Yes indeed, thank you for noticing that, I had to copy the code from my local install and forgot to change the fields to their individual names. It is fixed now.

Forgot to change the labels with the duplicate field fix commit. That is fixed now too.
@copacomilan
Copy link
Author

And that obviously goes for the labels for these filter fields as well, luckily I did notice that mistake myself upon review. Everything should be in order now.

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.

Filtering on date fields within Hardware Lifecycle view
2 participants