Parsely / Parsely/wp-parsely

PCI: Implement WordPress.com compatibility

Open
#2,882 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

wp-parsely
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/v1 isn’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

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.