Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FEATURE: Add DecendantOfNodetype Condition Generator #3403

Closed

Conversation

vcg-development
Copy link

This change adds the DecendantOfNodetypeConditionGenerator which will be used by the isDescendantOfNodetype privilege target matcher in the neos-development-collection repository.

Review instructions

A test for the isDescendantOfNodetype privilege target matcher can be found in the commit in the "neos-development-collection" repository.

Checklist

  • Code follows the PSR-2 coding style
  • Tests have been created, run and adjusted as needed
  • The PR is created against the lowest maintained branch
  • Reviewer - PR Title is brief but complete and starts with FEATURE|TASK|BUGFIX
  • Reviewer - The first section explains the change briefly for change-logs
  • Reviewer - Breaking Changes are marked with !!! and have upgrade-instructions

@github-actions github-actions bot added the 8.3 label Oct 16, 2024
@vcg-development vcg-development changed the title Add DecendantOfNodetype Condition Generator FEATURE: Add DecendantOfNodetype Condition Generator Oct 18, 2024
@kitsunet
Copy link
Member

Hi there, welcome and thank you for your contribution.

As this is a FEATURE it should only go into the next following minor or major release, as 8.3 is already released, this would be 8.4 now which is not yet released. So I will adjust the PR to target 8.4 instead.

@kitsunet kitsunet changed the base branch from 8.3 to 8.4 October 21, 2024 11:05
@github-actions github-actions bot added 8.4 and removed 8.3 labels Oct 21, 2024
@skurfuerst
Copy link
Member

Related: neos/neos-development-collection#5291

Copy link
Member

@skurfuerst skurfuerst left a comment

Choose a reason for hiding this comment

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

Hey,

Reviewed in neos/neos-development-collection#5291 (review) :) -> let's discuss there.
All the best,
Sebastian

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants