Skip to content

Placement Group Sync Workflow Action - #202

Merged
gmcgrady merged 4 commits into
masterfrom
sl-feature-camp-sync
Mar 5, 2026
Merged

Placement Group Sync Workflow Action#202
gmcgrady merged 4 commits into
masterfrom
sl-feature-camp-sync

Conversation

@stphnlee

@stphnlee stphnlee commented Mar 5, 2026

Copy link
Copy Markdown
Contributor

This workflow action is for syncing groupmember attribute values on registration groups when a member is added to or removed from a placement group associated with that registration instance.

Copilot AI review requested due to automatic review settings March 5, 2026 15:17

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds a new Rock RMS workflow action component to keep registration group member attribute values in sync with placement group membership changes (add/remove), supporting scenarios like camps where placement drives check-in/labels.

Changes:

  • Introduces UpdateRegistrationGroupWithPlacementGroup workflow action to update registration group member attributes based on placement group membership, with Lava-enabled configuration.
  • Updates the workflow plugin project file to compile the new action.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
Plugins/org.secc.Workflow/org.secc.Workflow.csproj Includes the new workflow action source file in the build.
Plugins/org.secc.Workflow/Registrations/UpdateRegistrationGroupWithPlacementGroup.cs Implements the workflow action and its placement-to-registration synchronization logic.

…al; added trim to groupnameprefix; updated logic for finding other matching placement groups to include groups attached to the template
@stphnlee
stphnlee requested a review from gmcgrady March 5, 2026 18:28

@gmcgrady gmcgrady left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This PR should effectively replace our SQL job for syncing placement groups & group member attributes in registration groups. Instance-level and template-level placement configurations makes it future-proof. Also, great catch on checking for placement in another group during a removal operation. The architecture of using a workflow triggered by the group type makes the 'Execute()' method of this action straight forward. Approving & merging.

@gmcgrady
gmcgrady merged commit 48df59e into master Mar 5, 2026
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.

3 participants