Taxon cover images and external references #940
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* feat: management command for updating fields on existing Taxa from CSV * feat: try another title for example occurrence image * feat: allow manual filter for showing taxa without occurrences * feat: allow sorting by reference image for better demos
* feat: add fields for Taxon cover images and Fieldguide ID * feat: add cover image fields to Taxon model for the UI
✅ Deploy Preview for antenna-preview ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
mihow
changed the title
…overs-and-references
There was a problem hiding this comment.
Pull Request Overview
This PR adds support for taxon cover images and external references (Fieldguide) by introducing new database fields and API endpoints. The changes enable species to have associated cover images with credits and fieldguide references for external taxonomy resources.
- Added database fields for fieldguide_id, cover_image_url, and cover_image_credit to the Taxon model
- Updated API serializers to expose the new fields in various endpoints
- Enhanced the TypeScript Species model with getters for the new fields
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| ami/main/models.py | Added three new fields to Taxon model for external references and cover images |
| ami/main/migrations/0074_taxon_cover_image_credit_taxon_cover_image_url_and_more.py | Database migration to add the new fields |
| ami/main/api/serializers.py | Updated serializers to include new fields in API responses |
| ami/main/api/views.py | Added cover_image_url to TaxonViewSet ordering fields |
| ui/src/data-services/models/species.ts | Added TypeScript getters for fieldguide_id, cover images, and refactored fieldguideUrl |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Fields for species cover images and external references (Fieldguide)
This is a remake of #871 and #822 that were added to the experimental #820 branch.
Follow-up PR with auto-selection of cover image: #832
I tested with our existing global species list with image fields & fieldguide IDs. It still works!
This has been tested more thoroughly in a previous environments. I am cherry picking this feature to main now.