Skip to content

Conversation

@andriacap
Copy link
Contributor

@andriacap andriacap commented Aug 12, 2025

Cette PR propose de pouvoir filtrer sur le champ "additional_data" des observations liées aux protocoles du module monitoring.
Voir détail sur l'issue #3662 .

@codecov
Copy link

codecov bot commented Aug 12, 2025

Codecov Report

❌ Patch coverage is 14.28571% with 42 lines in your changes missing coverage. Please review.
✅ Project coverage is 84.23%. Comparing base (8dc677e) to head (4e370ad).
⚠️ Report is 2 commits behind head on develop.

Files with missing lines Patch % Lines
...nature/core/gn_synthese/utils/query_select_sqla.py 9.37% 29 Missing ⚠️
backend/geonature/core/gn_commons/routes.py 20.00% 8 Missing ⚠️
backend/geonature/utils/config.py 28.57% 5 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #3663      +/-   ##
===========================================
- Coverage    84.56%   84.23%   -0.33%     
===========================================
  Files          133      133              
  Lines        10320    10367      +47     
===========================================
+ Hits          8727     8733       +6     
- Misses        1593     1634      +41     
Flag Coverage Δ
pytest 84.23% <14.28%> (-0.33%) ⬇️

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.

@andriacap andriacap force-pushed the feat/add-filter-additional-data-synthese branch 2 times, most recently from 04d4f37 to 4e370ad Compare August 19, 2025 16:27
- Call function from gn_module_monitoring to enrich monitoring config in order to get fiedls to filter
- Add subgroup in form object in order to target filters for monitoring

Reviewed-by: @andriacap
- Use config parameters "show_additional_filters" in order to display or not advanced filters monitorings

Reviewed-by: @andriacap
- need to adapt code in frontend and backend to use new config structure (object with protocol keys)

Reviewed-by: andriac
- Transform object date in order to compare with string date format
- Transform list concatenated data with ',' in json field additional_data to use contain fonction

Reviewed-by: acapai
@andriacap andriacap force-pushed the feat/add-filter-additional-data-synthese branch from 4e370ad to e4820d1 Compare August 29, 2025 14:13
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.

1 participant