Skip to content

Enhancement: Added better logging for retired user error#284

Merged
kalisp merged 1 commit intodevelopfrom
enhancement/YN-0460_update_exception_comments_addressings
Feb 4, 2026
Merged

Enhancement: Added better logging for retired user error#284
kalisp merged 1 commit intodevelopfrom
enhancement/YN-0460_update_exception_comments_addressings

Conversation

@kalisp
Copy link
Member

@kalisp kalisp commented Jan 30, 2026

Changelog Description

Synchronizing Comment with '@RETIRED_USER_IN_SG' from AYON triggers nondescript:

shotgun_api3.shotgun.Fault: API create() CRUD ERROR #6: Invalid field value, update failed [5 - Update failed for [Note.addressings_to]: Value is not legal.]

This PR logs it more verbosely with:

ERROR: Failed to create SG Note for AYON activity 5729ead0fddd11f09fae591c14fdf166: Comment contains link to retired userin SG in '[{'type': 'HumanUser', 'id': 253}, {'type': 'HumanUser', 'id': 156}]

Testing notes:

  1. retire account in SG that you have synchronized in AYON
image
  1. reference that account in @ in Comment
  2. unretire account from Trash > People

@kalisp kalisp requested review from MilaKudr and rdelillo January 30, 2026 13:20
@kalisp kalisp self-assigned this Jan 30, 2026
@kalisp kalisp added type: enhancement Improvement of existing functionality or minor addition sponsored This is directly sponsored by a client or community member labels Jan 30, 2026
@kalisp kalisp merged commit da34fd7 into develop Feb 4, 2026
7 checks passed
@kalisp kalisp deleted the enhancement/YN-0460_update_exception_comments_addressings branch February 4, 2026 11:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sponsored This is directly sponsored by a client or community member type: enhancement Improvement of existing functionality or minor addition

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants