PROD-31064: Replace node grants in Search API #4116
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
nechai
force-pushed
the
feature/replace-node-grants-search-api
branch
2 times, most recently
from
83935f7 to
4565a09
Compare
nechai
changed the title
nechai
changed the title
nechai
changed the title
nechai
force-pushed
the
feature/replace-node-grants-search-api
branch
2 times, most recently
from
908e520 to
26abf3b
Compare
nechai
force-pushed
the
feature/3458767-replace-node-grants
branch
from
3b2a0de to
b52e1bb
Compare
nechai
force-pushed
the
feature/replace-node-grants-search-api
branch
2 times, most recently
from
f22fe99 to
56ac1f2
Compare
nechai
force-pushed
the
feature/replace-node-grants-search-api
branch
from
56ac1f2 to
cda5ecd
Compare
tregismoreira
approved these changes
Nov 21, 2024
|
|
||
| $index->removeProcessor('content_access'); | ||
| \Drupal::messenger() | ||
| ->addWarning(t('"Content Access" processor is not allowed for using (replaced by query alters).')); |
There was a problem hiding this comment.
Suggested change
| ->addWarning(t('"Content Access" processor is not allowed for using (replaced by query alters).')); | |
| ->addWarning(t('"Content Access" processor is deprecated, and it's replaced by query alters.')); |
ribel
reviewed
Dec 20, 2024
| * "preprocess_query" = -30, | ||
| * }, | ||
| * locked = true, | ||
| * hidden = true, |
There was a problem hiding this comment.
@nechai as we discussed would be good to make at least this one visible from UI.
ribel
reviewed
Dec 20, 2024
| * | ||
| * @SearchApiProcessor( | ||
| * id = "social_search_tagging_query", | ||
| * label = @Translation("Tagging queries"), |
There was a problem hiding this comment.
Better to rename it to something more specific, like Social entity access
nechai
force-pushed
the
feature/3458767-replace-node-grants
branch
from
ec9b509 to
a882e33
Compare
ribel
changed the title
nechai
force-pushed
the
feature/replace-node-grants-search-api
branch
from
8024456 to
42b2c02
Compare
ribel
added
team: enterprise
These conditions groups allows developers to subscribe to each group (because it is tagged) and add own conditions or alters
…unity" visibilities
nechai
force-pushed
the
feature/replace-node-grants-search-api
branch
from
42b2c02 to
8b7e3c4
Compare
|
Tugboat has finished building the preview for this pull request! Link: Dashboard: |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Caution
PR should be merged in #3961
Note
This PR doesn't require behat tests passing. Tests should be checked in #3961
Replace
node grantsfor search API queries.More details about the reasons why we need this PR can be found at #4098