Skip to content
Merged
Show file tree
Hide file tree
Changes from 3 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions dojo/api_v2/views.py
Original file line number Diff line number Diff line change
Expand Up @@ -1099,6 +1099,9 @@ def notes(self, request, pk=None):
note_type=note_type,
)
note.save()
finding.last_reviewed = note.date
finding.last_reviewed_by = author
finding.save(update_fields=["last_reviewed", "last_reviewed_by"])
Comment thread
Maffooch marked this conversation as resolved.
Outdated
finding.notes.add(note)
# Determine if we need to send any notifications for user mentioned
process_tag_notifications(
Expand Down
4 changes: 3 additions & 1 deletion dojo/jira_link/views.py
Original file line number Diff line number Diff line change
Expand Up @@ -285,7 +285,9 @@
finding.notes.add(new_note)
finding.jira_issue.jira_change = timezone.now()
finding.jira_issue.save()
finding.save()
finding.last_reviewed = note.date

Check failure on line 288 in dojo/jira_link/views.py

View workflow job for this annotation

GitHub Actions / ruff-linting

Ruff (F821)

dojo/jira_link/views.py:288:37: F821 Undefined name `note`

Check failure on line 288 in dojo/jira_link/views.py

View workflow job for this annotation

GitHub Actions / ruff-linting

Ruff (F821)

dojo/jira_link/views.py:288:37: F821 Undefined name `note`
Comment thread
Maffooch marked this conversation as resolved.
Outdated
finding.last_reviewed_by = author
finding.save(update_fields=["last_reviewed", "last_reviewed_by"])
Copy link
Copy Markdown
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

return None


Expand Down
Loading