PCI: Implement WordPress.com compatibility
Nobody has claimed this yet.
- Dominant language
- PHP
- Stars
- 66
- Forks
- 36
- Avg merge
- 2d 2h
- Merged PRs (30d)
- 16
Description
Is your feature request related to a problem?
Our PCI functionality is currently not compatible with WordPress.com, due to the way our WP Rest API is structured.
To quote @travisw:
It appears the way certain API related code is currently implemented in the plugin, might be causing issues when running on WordPress.com. Specifically, registering the top-level namespace
wp-parsely/v1isn’t allowed like it might be on standalone WordPress installs.
We might need to think about modifying the plugin code, to allow the API namespace to be varied depending on if it’s running on WordPress.com or not. It’d need to be done both on the back-end (when registering endpoints) and on the front-end (when making API requests).
Describe the solution you'd like
@travisw has also provided some proof of concept code to us, which verifies that this solution should work. As such, we just need to see how we would implement this in the plugin, both on the back-end and front-end.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating where the plugin registers the wp-parsely/v1 REST API namespace and where the front end constructs requests to it. Review the referenced proof-of-concept code if available and trace both registration and request paths; done means the PCI functionality works with the varied namespace on WordPress.com without breaking standalone WordPress.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- php, wordpress
- Domain
- api, backend
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100