Skip to content

Conversation

@bwl21
Copy link
Owner

@bwl21 bwl21 commented Sep 27, 2025

Follow-up to PR #24

This PR adds the missing documentation and permission adjustments that were created after the initial automatic groups grouptype fix was merged.

Changes

Documentation

  • Complete development session documentation with technical decisions
  • Add lessons learned about ChurchTools API patterns and data structures
  • Document ID-to-name mapping strategy for future reference

Permission Fix

  • Adjust automatic groups permission from 'churchdb' + 'administer groups' to 'churchgroup' + 'view'
  • Makes the module accessible to more users while maintaining security
  • Users only need to view groups to see automatic group information, not administer them

Commits

  • 53c0cfb: docs: add session documentation for automatic groups grouptype fix
  • de3e65b: fix: adjust automatic groups permission requirements

These commits were created after PR #24 was merged and contain important documentation and permission improvements.

bwl21 and others added 3 commits September 27, 2025 13:49
- Complete development session documentation with technical decisions
- Add lessons learned about ChurchTools API patterns and data structures
- Document ID-to-name mapping strategy for future reference

Co-authored-by: Ona <no-reply@ona.com>
Change from 'churchdb' + 'administer groups' to 'churchgroup' + 'view'
to make the module accessible to more users while maintaining security.

The previous permission was too restrictive - users only need to view
groups to see automatic group information, not administer them.

Co-authored-by: Ona <no-reply@ona.com>
- Remove trailing whitespace
- Add missing newline at end of file

Co-authored-by: Ona <no-reply@ona.com>
@bwl21 bwl21 merged commit 41e4fc5 into main Sep 27, 2025
1 check passed
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.

2 participants