fix(config_guide): correct CEL and variables docs against codebase#131
fix(config_guide): correct CEL and variables docs against codebase#131emjay0921 wants to merge 1 commit intov2-in-progress-3from
Conversation
|
There was a problem hiding this comment.
Code Review
This pull request updates the OpenSPP documentation to reflect workflow changes, including renaming 'publishing' to 'activating' and updating menu navigation. It also transitions event data access from aggregate targets to dedicated CEL functions and simplifies string function recommendations. A review comment correctly identifies a need to remove the 'events' relation from the syntax guide's relations table to maintain consistency with these architectural changes.
| | `registry_individuals` | Individual registrant | `groups`, `enrollments`, `entitlements`, `events` | | ||
| | `registry_groups` | Group/household | `members`, `enrollments`, `entitlements`, `events` | |
There was a problem hiding this comment.
The events relation should be removed from the 'Available Relations' column to maintain consistency with the rest of the documentation changes. The PR description and other updated files (e.g., variables.md, variable_types.md) state that event data is no longer an aggregate target and should be accessed via dedicated event functions instead of collection relations.
| | `registry_individuals` | Individual registrant | `groups`, `enrollments`, `entitlements`, `events` | | |
| | `registry_groups` | Group/household | `members`, `enrollments`, `entitlements`, `events` | | |
| | registry_individuals | Individual registrant | groups, enrollments, entitlements | | |
| | registry_groups | Group/household | members, enrollments, entitlements | |



Why is this change needed?
QA review found 14 inaccuracies in the CEL and variables documentation when verified against the OpenSPP2 codebase.
How was the change implemented?
whereparameter and Python fallback referencesRelated links
https://projects.acn.fr/projects/acn-eng/work_packages/823