Automattic / Automattic/jetpack

Change from the Calypso side and remove the special behaviour for search from `class.wpcom-json-api.php`

Open
#24,447 0 comments 0 reactions 0 assignees View on GitHub
[Feature] Search [Package] Search [Plugin] Search
Dominant language
PHP
Stars
1.8k
Forks
898
Avg merge
1d 18h
Merged PRs (30d)
774

Description

Previously we had some special behaviour coded to `class.wpcom-json-api.php ` for a quick fix for the `rest/1.2/sites/xxx/settings`, but probably we need to leverage our new api rather and change in Calypso rather than tweaking the legacy code too much.

Related:
- p9dueE-55u#comment-7974
- p9dueE-55u#comment-7987
- D78351-code

Contributor guide

Open the contributing guide

Research direction

Review class.wpcom-json-api.php and the behavior around rest/1.2/sites/xxx/settings, then read the related p9dueE-55u comments and D78351-code. Determine how the new API and Calypso should handle the search behavior, and consider the work complete when the legacy special behavior is removed from this file and the change is handled on the Calypso side.

Written by the indexing model from the issue text.

Assessment

Tech stack
php
Domain
api, backend
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
20/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.