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

[CMSP-579] Pass more info to pantheon_wp_rest_api_surrogate_keys filter #235

Open
TrilipuT opened this issue Jun 8, 2023 · 1 comment
Open

Comments

@TrilipuT
Copy link

TrilipuT commented Jun 8, 2023

Hi
I tried to add surrogate keys to my custom REST endpoint but found that it's a bit tricky to define what requests you are on right now. I used $_SERVER['REQUEST_URI'] to check what uri is being requested.
I think it will be easier if you can pass more parameters from original filter - that will help to operate data more flexible.
$response, $server, $request or whatewer available there.

@jazzsequence
Copy link
Contributor

jazzsequence commented Aug 8, 2023

Thanks for the feedback. Tracking this internally as CMSP-579. We'll need to figure out where to prioritize this and add it into our development schedule, but it's a valid point and would be a good add.

@jazzsequence jazzsequence changed the title Pass more info to pantheon_wp_rest_api_surrogate_keys filter [CMSP-579] Pass more info to pantheon_wp_rest_api_surrogate_keys filter Aug 8, 2023
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

No branches or pull requests

2 participants