Skip to content

Conversation

ad-m-ss
Copy link

@ad-m-ss ad-m-ss commented Sep 15, 2025

Summary

Fixes error from trying to update fields that don't exist in the database.

post_migrate signal received apps signal. That has the model state after migration applied:

image

This is especially important for rollback.

Test plan

  • Existing tests continue to pass
  • New test case validates inherited field handling
  • Syntax validation passes

This resolves the issue originally addressed in PR #25 which was closed due to the deleted branch.

🤖 Generated with Claude Code

ad-m-ss and others added 2 commits September 15, 2025 09:01
- Add get_existing_fields() function to check database schema via introspection
- Modify get_comments_for_model() to only process fields that exist in the database
- Add test for model inheritance scenarios
- Fixes errors when updating comments for inherited model fields

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@ad-m-ss ad-m-ss changed the title [Fix] Skip inherited fields that don't exist in database [Fix] Skip deleted fields that don't exist in database Sep 15, 2025
@ad-m-ss ad-m-ss changed the title [Fix] Skip deleted fields that don't exist in database [Fix] Unblock rollback operation Sep 15, 2025
@ad-m-ss ad-m-ss changed the title [Fix] Unblock rollback operation fix: Unblock rollback operation Sep 15, 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.

1 participant