Skip to content

feat: enhance property setter migration with smart update logic#43

Merged
aakvatech merged 1 commit into
Aakvatech-Limited:version-15from
iamtalib13:feat/add-under-construction-status
Feb 11, 2026
Merged

feat: enhance property setter migration with smart update logic#43
aakvatech merged 1 commit into
Aakvatech-Limited:version-15from
iamtalib13:feat/add-under-construction-status

Conversation

@iamtalib13
Copy link
Copy Markdown
Contributor

@iamtalib13 iamtalib13 commented Feb 11, 2026

Overview
This PR improves the robustness of the create_property_setter utility script to ensure efficient and accurate synchronization of property setters.

Changes

  • Infrastructure: Refactored propms/utils/create_property_setter.py to implement a Smart Update mechanism.
  • Composite Key Check: Replaced simple name-based skipping with a composite key validation (doc_type, field_name, property).
  • Value Normalization: Added logic to compare existing database values with JSON values, ensuring updates only trigger when an actual change is detected.
  • Optimization: Prevents redundant database writes and ensures all JSON-defined setters are correctly applied during bench migrate.

Impact

  • Faster and more reliable migrations.
  • Ensures property setter changes in JSON files are always in sync with the database.

Testing

  • Verified via bench migrate: Confirmed that existing setters are skipped and only modified values are updated.

@iamtalib13 iamtalib13 changed the title feat: add 'Under Construction' status and enhance property setter migration workflow feat: enhance property setter migration with smart update logic Feb 11, 2026
@iamtalib13 iamtalib13 force-pushed the feat/add-under-construction-status branch from 83cb1c1 to c7c078d Compare February 11, 2026 19:46
@aakvatech aakvatech merged commit 5d6aecc into Aakvatech-Limited:version-15 Feb 11, 2026
1 check passed
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