Skip to content
Snippets Groups Projects

Data Layer updates a "last edit date" field on the Item

Merged decentral1se requested to merge last_modified into develop

Created by: martinburchell

Merge request reports

Approval is optional

Merged by avatar (Apr 15, 2025 8:44am UTC)

Merge details

  • Changes merged into with fd47d1e7.
  • Deleted the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Author Contributor

    Created by: aliceh75

    I think it would be good to have a test that ensures that modifying one item does not update the last modified time on another item.

    This would also ensure that the time mocking thing is functioning - at the moment, all the tests check for a single update, so if num_updates was broken and always returned 1 all the tests would still pass.

  • Author Contributor

    Created by: martinburchell

    Good idea - added

    num_updates was returning values other than 1 during development, before I put the action checks in the signal handler.

Please register or sign in to reply
Loading