Skip to content

Conversation

mihow
Copy link
Collaborator

@mihow mihow commented Sep 16, 2025

Summary

  • Add missing migrations for job model
  • Update default setting to resolve warning

@Copilot Copilot AI review requested due to automatic review settings September 16, 2025 02:02
Copy link

netlify bot commented Sep 16, 2025

Deploy Preview for antenna-preview canceled.

Name Link
🔨 Latest commit 8f54c2b
🔍 Latest deploy log https://app.netlify.com/projects/antenna-preview/deploys/68c9d637a5664600095e2a63

Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR addresses Django migration issues by adding missing migrations for the job model and updating a default setting to resolve a warning.

  • Adds migration file to alter job model's logs and progress fields
  • Updates ProjectFeatureFlags default from function reference to class reference

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
ami/main/models.py Updates default parameter for feature_flags field from function to class
ami/jobs/migrations/0017_alter_job_logs_alter_job_progress.py Adds Django migration to alter job logs and progress fields with proper Pydantic schema configuration

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@mihow mihow requested a review from Copilot September 16, 2025 20:41
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@mihow mihow merged commit 83d2a04 into main Sep 16, 2025
6 checks passed
@mihow mihow deleted the fix/missing-migrations branch September 16, 2025 21:40
mihow added a commit that referenced this pull request Sep 17, 2025
this is only a configuration change, it does not alter the DB schema
mihow added a commit that referenced this pull request Sep 17, 2025
* Import cover images & external references (#838)

* 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

* Fields for Taxon reference images (#822)

* feat: add fields for Taxon cover images and Fieldguide ID

* feat: add cover image fields to Taxon model for the UI

* fix: look up existing taxa by name only

* fix: update existing genus parents if their rank is wrong, don't crash

* fix: don't allow empty CSV columns to clear existing data

* fix: ensure all taxa in import list are added to taxa list in DB

* fix: only keep changes related to importing taxa in this branch

* Apply suggestions from code review

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* fix: missing migration after last commit in #946

this is only a configuration change, it does not alter the DB schema

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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