Skip to content

Set last reviewed date and reviewer when note is added#14209

Merged
Maffooch merged 7 commits intobugfixfrom
Maffooch-patch-1
Feb 4, 2026
Merged

Set last reviewed date and reviewer when note is added#14209
Maffooch merged 7 commits intobugfixfrom
Maffooch-patch-1

Conversation

@Maffooch
Copy link
Contributor

Update finding's last reviewed date and reviewer to maintain parity with UI behaviors

Update finding's last reviewed date and reviewer to maintain parity with UI behaviors
@Maffooch Maffooch requested a review from mtesauro as a code owner January 29, 2026 20:42
@github-actions github-actions bot added the apiv2 label Jan 29, 2026
@Maffooch Maffooch added this to the 2.55.0 milestone Jan 29, 2026
@valentijnscholten valentijnscholten changed the title Set last reviewed date and reviewer for finding Set last reviewed date and reviewer when note is added Jan 29, 2026
@valentijnscholten
Copy link
Member

Should this also happen on notes added via the JIRA webhook?

Copy link
Contributor

@mtesauro mtesauro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved

@Maffooch
Copy link
Contributor Author

Should this also happen on notes added via the JIRA webhook?

Sure

Update finding with last reviewed date and author.
finding.save()
finding.last_reviewed = new_note.date
finding.last_reviewed_by = author
finding.save(update_fields=["last_reviewed", "last_reviewed_by"])
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think "updated" should also be included

Copy link
Contributor

@blakeaowens blakeaowens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved pending Val's comment.

@Maffooch Maffooch closed this Feb 2, 2026
@Maffooch Maffooch reopened this Feb 2, 2026
@Maffooch
Copy link
Contributor Author

Maffooch commented Feb 2, 2026

Turns out that updated is not a valid field on the model, but it will be shortly once #14198 lands. Will sit on this PR so that tests can pass

@Maffooch Maffooch modified the milestones: 2.55.0, 2.55.1 Feb 2, 2026
@Maffooch Maffooch changed the base branch from bugfix to dev February 2, 2026 19:29
@Maffooch Maffooch changed the base branch from dev to bugfix February 2, 2026 19:29
@Maffooch Maffooch merged commit 9e651dc into bugfix Feb 4, 2026
150 checks passed
@Maffooch Maffooch deleted the Maffooch-patch-1 branch February 4, 2026 22:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants