You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I think it would be good to implement this style for WPGraphQL Smart Cache as it makes things more readable (at least in my opinion) and I can more clearly understand the "domain" the hook is related to.
Something like:
wpgraphql/smart-cache/$action_name
or
wpgraphql/cache/$action_name
(not 💯 sold either way on "smart-cache" vs "cache" 🤷🏻♂️ )
currently the codebase has actions and filters with mixed naming conventions:
Some start with
wpgraphql_cache
(such aswpgraphql_cache_save_request
) and some start withgraphql_cache
such asgraphql_purge
).We should decide which prefix to use and make things consistent.
Note, we'll need to keep around the old actions/filters and deprecate them as folks could already be using them.
Some helpful info re: deprecating in here: https://juagonala.com/how-to-deprecate-code-in-wordpress/
The text was updated successfully, but these errors were encountered: