Skip to content

#6200 - Let users see their own roles in a project via remote API - #6208

Merged
reckart merged 1 commit into
mainfrom
feature/6200-Let-users-see-their-own-roles-in-a-project-via-remote-API
Aug 3, 2026
Merged

#6200 - Let users see their own roles in a project via remote API#6208
reckart merged 1 commit into
mainfrom
feature/6200-Let-users-see-their-own-roles-in-a-project-via-remote-API

Conversation

@reckart

@reckart reckart commented Aug 2, 2026

Copy link
Copy Markdown
Member

What's in the PR

  • Convert RProject from a mutable class to a record with a builder, enabling deserialization by Jackson without additional constructors
  • Add explicit PermissionLevel role arguments to getProject(...) and createCompatibleCas(...) calls across all AERO remote API controllers (AeroAnnotationController, AeroCurationController, AeroDocumentController, AeroKnowledgeBaseController, AeroProjectController, LegacyRemoteApiController, and related Controller_ImplBase/next/* classes)
  • Update test helper MockAeroClient to deserialize and return RProject objects instead of extracting IDs via jsonPath
  • Convert AeroProjectControllerTest call sites to use project object references instead of extracting IDs inline, eliminating hardcoded project ID assumptions that break when the database identity sequence is non-sequential
  • Add LegacyRemoteApiControllerTest (new file)
  • Clean up static import usage and logger modifier (private -> private static)

How to test manually

  • See issue description

Automatic testing

  • PR includes unit tests

Documentation

  • PR updates documentation

@reckart reckart added this to the 42.0 milestone Aug 2, 2026
@reckart reckart self-assigned this Aug 2, 2026
@reckart reckart added the ⭐️ Enhancement New feature or request label Aug 2, 2026
@reckart reckart added this to Kanban Aug 2, 2026
@github-project-automation github-project-automation Bot moved this to 🔖 To do in Kanban Aug 2, 2026
- Convert RProject from a mutable class to a record with a builder, enabling deserialization by Jackson without additional constructors
- Add explicit PermissionLevel role arguments to getProject(...) and createCompatibleCas(...) calls across all AERO remote API controllers (AeroAnnotationController, AeroCurationController, AeroDocumentController, AeroKnowledgeBaseController, AeroProjectController, LegacyRemoteApiController, and related Controller_ImplBase/next/* classes)
- Update test helper MockAeroClient to deserialize and return RProject objects instead of extracting IDs via jsonPath
- Convert AeroProjectControllerTest call sites to use project object references instead of extracting IDs inline, eliminating hardcoded project ID assumptions that break when the database identity sequence is non-sequential
- Add LegacyRemoteApiControllerTest (new file)
- Clean up static import usage and logger modifier (`private` -> `private static`)
@reckart
reckart force-pushed the feature/6200-Let-users-see-their-own-roles-in-a-project-via-remote-API branch from 2c8608a to 2289758 Compare August 2, 2026 20:19
@reckart
reckart merged commit 59f9c0b into main Aug 3, 2026
5 of 6 checks passed
@reckart
reckart deleted the feature/6200-Let-users-see-their-own-roles-in-a-project-via-remote-API branch August 3, 2026 20:41
@github-project-automation github-project-automation Bot moved this from 🔖 To do to 🍹 Done in Kanban Aug 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: 🍹 Done

Development

Successfully merging this pull request may close these issues.

1 participant