Skip to content

GRUD_DEV-1250/group column permissions - #333

Merged
smnhgn merged 2 commits into
masterfrom
GRUD_DEV-1250/group-column-permissions
Sep 14, 2026
Merged

smnhgn merged 2 commits into
masterfrom
GRUD_DEV-1250/group-column-permissions

Conversation

@smnhgn

@smnhgn smnhgn commented Sep 11, 2026 •

Copy link
Copy Markdown
Member

Submit a pull request

Related Ticket: GRUD_DEV-1250

Please make sure the following is true

  • I gave the PR a meaningful name
  • I checked that the correct target branch is selected
  • I rebased the branch on the target branch and it can be merged
  • I ran the linter and it did pass
  • I checked for unused code / dead code / debug code
  • I checked that variables/functions have meaningful names
  • I checked that the behaviour is as the documentation/task describes and I tested it
  • I updated the docs / specifications if possible
  • I could explain all that code when someone wakes me up at 3am
  • I checked that the code considers failures and not just the happy path
  • There are no new dependencies OR I listed them and explained them below
  • PR introduces no breaking changes OR I listed them and described them below
  • I added/updated tests for new/modified unit-testable functions/helpers
  • I ran the tests and they did pass

Other information/comments (e.g. reasons why points are not checked from above)

Reason for this PR

Anpassungen zur Group-Spalte:

  • Entfernt Feld concats aus Group-Spalte
    Bisher enthielt eine Group-Spalte sowohl concats (mit permission) und groups (ohne permissions)
  • Fügt den Spalten in groups permissions hinzu

@smnhgn
smnhgn requested a review from zingmane September 11, 2026 11:14

@zingmane zingmane left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Evtl. Wäre ein Test noch ganz gut. So sieht's aber gut aus

@github-actions

github-actions Bot commented Sep 11, 2026 •

Copy link
Copy Markdown

Test Results

1 191 tests  +1 191   1 179 ✅ +1 179   3m 55s ⏱️ + 3m 55s
  146 suites +  146      12 💤 +   12 
  146 files   +  146       0 ❌ ±    0 

Results for commit 39e39e1. ± Comparison against base commit 11f54d3.

♻️ This comment has been updated with latest results.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@smnhgn
smnhgn merged commit 462f517 into master Sep 14, 2026
10 of 11 checks passed
@smnhgn
smnhgn deleted the GRUD_DEV-1250/group-column-permissions branch September 14, 2026 08:58
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