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

Issue #3458767: Replace node grants for "public" and "community" visibilities #4109

Conversation

nechai
Copy link
Contributor

@nechai nechai commented Oct 9, 2024

Caution

PR should be merged in #3961

Note

This PR doesn't require behat tests passing. Tests should be checked in #3961

Replace node grants with query alters for nodes with public and community visibilities.
More details about the reasons why we need this PR can be found at #4098

@nechai nechai self-assigned this Oct 9, 2024
Copy link

github-actions bot commented Oct 9, 2024

Thanks for opening this Pull Request! To ensure Pull Requests are easy to find and understand for all our team-members we've established some standardised rules. While reviewing this pull request I've found the following issues, please resolve them.

Missing milestone
The issue is missing a milestone. Milestones are important to help release managers know whether they might need to postpone a release for a critical issue and to communicate what fixes and new features are included in a release. Before merging, please select the appropriate milestone for this pull request.

🤖 This is an automatically produced message by the Open Social PR manager.

@nechai nechai force-pushed the feature/replace-node-grants-visibilities branch 3 times, most recently from 374c034 to 7e09375 Compare October 14, 2024 13:31
@nechai nechai added type: refactoring Updates code for improved maintenance without changing its functionality team: enterprise This PR originates from the ECI team type: feature Adds a new feature to Open Social prio: medium labels Oct 15, 2024
@nechai nechai force-pushed the feature/replace-node-grants-visibilities branch from 7e09375 to 1e1f77c Compare October 15, 2024 13:25
@nechai nechai force-pushed the feature/3458767-replace-node-grants branch from 3b2a0de to b52e1bb Compare November 7, 2024 11:14
@nechai nechai force-pushed the feature/replace-node-grants-visibilities branch from 1e1f77c to edf0079 Compare November 7, 2024 11:29
Copy link
Contributor

@tregismoreira tregismoreira left a comment

Choose a reason for hiding this comment

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

LGTM

@nechai nechai force-pushed the feature/3458767-replace-node-grants branch from ec9b509 to a882e33 Compare December 20, 2024 10:18
@nechai nechai force-pushed the feature/replace-node-grants-visibilities branch from edf0079 to b9402af Compare December 20, 2024 10:19
@ribel ribel changed the title Replace node grants for "public" and "community" visibilities PROD-21807: Replace node grants for "public" and "community" visibilities Dec 20, 2024
@ribel ribel changed the title PROD-21807: Replace node grants for "public" and "community" visibilities Issue #3458767: Replace node grants for "public" and "community" visibilities Dec 20, 2024
@ribel ribel added status: needs review This pull request is waiting for a requested review and removed type: feature Adds a new feature to Open Social labels Dec 20, 2024
@ribel ribel merged commit 2539e06 into feature/3458767-replace-node-grants Dec 20, 2024
176 of 197 checks passed
@ribel ribel deleted the feature/replace-node-grants-visibilities branch December 20, 2024 11:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
prio: medium status: needs review This pull request is waiting for a requested review team: enterprise This PR originates from the ECI team type: refactoring Updates code for improved maintenance without changing its functionality
Development

Successfully merging this pull request may close these issues.

3 participants