Skip to content

Feat/add family filter#667

Open
TheoLechemia wants to merge 4 commits intodevelopfrom
feat/add_family_filter
Open

Feat/add family filter#667
TheoLechemia wants to merge 4 commits intodevelopfrom
feat/add_family_filter

Conversation

@TheoLechemia
Copy link
Member

Ajoute un filtre par famille sur taxref

@TheoLechemia TheoLechemia changed the base branch from master to develop January 8, 2026 10:09
@TheoLechemia TheoLechemia force-pushed the feat/add_family_filter branch from 2f08832 to 498f072 Compare January 8, 2026 16:29
@codecov
Copy link

codecov bot commented Jan 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 54.21%. Comparing base (6e5d283) to head (4510e98).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #667      +/-   ##
===========================================
+ Coverage    53.96%   54.21%   +0.25%     
===========================================
  Files           39       39              
  Lines         2763     2763              
===========================================
+ Hits          1491     1498       +7     
+ Misses        1272     1265       -7     
Flag Coverage Δ
pytest 54.21% <ø> (+0.25%) ⬆️

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.

@TheoLechemia
Copy link
Member Author

J'ai rajouté un test sur le filtre ajouté, je sais pas trop pourquoi le codecov a baissé ?

Copy link
Contributor

@jacquesfize jacquesfize left a comment

Choose a reason for hiding this comment

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

Il faudrait trouver un moyer de récupérer les index des columns. Un simple indexOf ?

sort_desc=None,
search=None,
# WARNING : le premier element du tuple est l'indice du tableau `column_filters` de la class Admin -> volatile !
filters=[(5, "Famille", "Arachnidiidae")],
Copy link
Contributor

Choose a reason for hiding this comment

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

passer par indexOf sur l'attribut column_filters pour récupérer cette index ?

@amandine-sahl amandine-sahl force-pushed the feat/add_family_filter branch from b325adf to 5a7ddc6 Compare February 6, 2026 14:06
@amandine-sahl amandine-sahl added this to the 2.2.4 milestone Feb 6, 2026
for tax in results:
assert tax.famille == "Arachnidiidae"

def test_filter_familly(self):
Copy link
Contributor

Choose a reason for hiding this comment

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

test_filter_familly a renommer en test_filter_order

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.

3 participants