Skip to content

Conversation

mohamedelabbas1996
Copy link
Contributor

@mohamedelabbas1996 mohamedelabbas1996 commented Sep 15, 2025

Summary

This PR introduces the project’s default taxa filter to the backend API. The filter is now applied consistently across OccurrenceViewSet, TaxonViewSet, SummaryView, and SourceImageViewSet, ensuring that all queries respect project-level default taxa settings.

List of Changes

TBD

Related Issues

#927

Detailed Description

TBD

How to Test the Changes

TBD

Screenshots

TBD

Deployment Notes

TBD

Checklist

  • I have tested these changes appropriately.
  • I have added and/or modified relevant tests.
  • I updated relevant documentation or comments.
  • I have verified that this PR follows the project's coding standards.
  • Any dependent changes have already been merged to main.

annavik and others added 20 commits August 28, 2025 10:41
…t, TaxonViewSet, SourceImageViewSet, EventViewSet, and SummaryView
Remove OccurrenceDeterminationScoreFilter to ensure apply_defaults parameter is respected.
Copy link

netlify bot commented Sep 15, 2025

Deploy Preview for antenna-preview canceled.

Name Link
🔨 Latest commit 22d7c48
🔍 Latest deploy log https://app.netlify.com/projects/antenna-preview/deploys/68c967513ac25700081866fb

@annavik
Copy link
Member

annavik commented Sep 16, 2025

@mohamedelabbas1996 with the separated approach, I think we should rebase this on #944. Let me know if I can help! :)

@mohamedelabbas1996
Copy link
Contributor Author

@mohamedelabbas1996 with the separated approach, I think we should rebase this on #944. Let me know if I can help! :)

@mohamedelabbas1996 with the separated approach, I think we should rebase this on #944. Let me know if I can help! :)

@annavik I’d really appreciate your help with the rebase. I tried doing it myself but ran into some conflicts in the frontend code

@annavik
Copy link
Member

annavik commented Sep 16, 2025

Closing in favor of #951

@annavik annavik closed this Sep 16, 2025
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