Skip to content

Integrate sync_schema_config_fields fixes into main#8067

Open
acwhite211 wants to merge 2 commits into
mainfrom
issue-8065
Open

Integrate sync_schema_config_fields fixes into main#8067
acwhite211 wants to merge 2 commits into
mainfrom
issue-8065

Conversation

@acwhite211
Copy link
Copy Markdown
Member

@acwhite211 acwhite211 commented May 8, 2026

Fixes #8065

Integrate sync_schema_config_fields fixes into main.

This ports the release branch schema config fix into main while preserving the newer main-branch schema config locking and de-duplication logic.

Checklist

  • Self-review the PR after opening it to make sure the changes look good and
    self-explanatory (or properly documented)
  • Add relevant issue to release milestone
  • Add pr to documentation list

Testing instructions

  • Run ve/bin/python manage.py sync_schema_config_fields --discipline-id <discipline_id>.
  • Confirm the “Missing fields” output does not include internal ID fields like accessionId, localityId, collectionObjectId, or other {table}Id fields.
  • Confirm legitimate missing non-ID fields still appear in the output.
  • Run ve/bin/python manage.py sync_schema_config_fields --discipline-id <discipline_id> --apply.
  • Confirm created schema config records are limited to expected non-ID fields.
  • Re-run the command and confirm previously created fields are no longer reported as missing.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 8, 2026

Warning

Rate limit exceeded

@acwhite211 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 31 minutes and 3 seconds before requesting another review.

You’ve run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: dc0c5086-c729-409c-9491-5f7c45311269

📥 Commits

Reviewing files that changed from the base of the PR and between b3a6255 and 6bc3377.

📒 Files selected for processing (2)
  • specifyweb/specify/migration_utils/update_schema_config.py
  • specifyweb/specify/models_utils/load_datamodel.py
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch issue-8065

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@acwhite211 acwhite211 added this to the 7.12.1 milestone May 18, 2026
@acwhite211 acwhite211 marked this pull request as ready for review May 18, 2026 21:55
@acwhite211 acwhite211 requested review from a team May 18, 2026 21:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 📋Back Log

Development

Successfully merging this pull request may close these issues.

Prevent sync_schema_config_fields from creating schema records for {table}Id fields

1 participant