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_access" in custom views filters #4115

Conversation

nechai
Copy link
Contributor

@nechai nechai commented Oct 15, 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_access table queries with node query access in custom views filters.
More details about the reasons why we need this PR can be found at #4098

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

github-actions bot commented Oct 15, 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/remove-node-grants-alter-views-filters branch 2 times, most recently from 4811298 to 40112ad Compare October 18, 2024 14:16
@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 18, 2024
@nechai nechai marked this pull request as ready for review October 18, 2024 14:18
@nechai nechai force-pushed the feature/remove-node-grants-alter-views-filters branch 2 times, most recently from 41c01eb to 61f228a Compare October 18, 2024 14:34
@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/3458767-replace-node-grants branch from ec9b509 to a882e33 Compare December 20, 2024 10:18
@nechai nechai force-pushed the feature/remove-node-grants-alter-views-filters branch from 61f228a to 6fa04d1 Compare December 20, 2024 10:28
@ribel ribel changed the title Replace "node_access" in custom views filters Issue #3458767: Replace "node_access" in custom views filters 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 d11775a into feature/3458767-replace-node-grants Dec 20, 2024
128 of 197 checks passed
@ribel ribel deleted the feature/remove-node-grants-alter-views-filters branch December 20, 2024 12:01
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.

2 participants