Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat/set audit fields via trigger #2759

Draft
wants to merge 20 commits into
base: develop
Choose a base branch
from

Conversation

dleard
Copy link
Contributor

@dleard dleard commented Jan 30, 2025

There are a ton of files changed in this PR. Most of the changes are either adding triggers to models by inheriting the Meta of the Timestamped Model or removing the set_create_or_update() function calls that have been replaced by the triggers.

The "meat" of the work done in this PR to apply triggers is in this one file:

  • registration/models/time_stamped_model.py

Most of the relevant tests are in these files:

  • conftest.py (one file in each moduel's "test" directory)
  • common/tests/utils/helpers.py
  • registration/tests/models/test_timestamped_model.py

Bug fix in:

  • service/user_operator_service_v2.py

@dleard dleard force-pushed the feat/set_audit_fields_via_trigger branch 2 times, most recently from d835da8 to 2c7e19a Compare January 31, 2025 19:27
@dleard dleard force-pushed the feat/set_audit_fields_via_trigger branch from fd921d0 to 05cc8f1 Compare January 31, 2025 23:40
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