Skip to content

fix: deserialize empty-array permissions in translator roles - #414

Merged
andrii-bodnar merged 1 commit into
masterfrom
fix/translator-role-empty-permissions
Sep 24, 2026
Merged

andrii-bodnar merged 1 commit into
masterfrom
fix/translator-role-empty-permissions

Conversation

@andrii-bodnar

Copy link
Copy Markdown
Member

Project members whose role has no per-language permissions are returned with "languagesAccess": [] (and sometimes "permissions": []) instead of an object, which failed with "Cannot deserialize value of type LinkedHashMap<String, LanguageAccessRule> from Array value".

  • annotate TranslatorRolePermissions.languagesAccess and TranslatorRole.permissions with EmptyArrayToNullDeserializer
  • make EmptyArrayToNullDeserializer read with the full JavaType so generic parameters are preserved (previously Map values came back as LinkedHashMap instead of the declared value type)
  • add language_coordinator, developer and manager to TranslatorRoleName so those role names no longer fail the whole response

Project members whose role has no per-language permissions are returned
with "languagesAccess": [] (and sometimes "permissions": []) instead of
an object, which failed with "Cannot deserialize value of type
LinkedHashMap<String, LanguageAccessRule> from Array value".

- annotate TranslatorRolePermissions.languagesAccess and
  TranslatorRole.permissions with EmptyArrayToNullDeserializer
- make EmptyArrayToNullDeserializer read with the full JavaType so
  generic parameters are preserved (previously Map values came back as
  LinkedHashMap instead of the declared value type)
- add language_coordinator, developer and manager to TranslatorRoleName
  so those role names no longer fail the whole response

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@andrii-bodnar
andrii-bodnar merged commit d9b69e9 into master Sep 24, 2026
2 checks passed
@andrii-bodnar
andrii-bodnar deleted the fix/translator-role-empty-permissions branch September 24, 2026 06:19
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