-
Notifications
You must be signed in to change notification settings - Fork 32
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
Refactor all endpoints code #2426
Comments
@vaurdan, tagging you here for notifications, in case you're not receiving any from this thread. |
I think there could be 2 main approaches here:
I'd probably suggest option 2, but happy to hear if there are different opinions. |
Things to think about, in no particular order:
There might be other things I've missed, let's put them all together in this thread. |
There's also an effort and private conversation to make PCH work with WordPress.com, which involves some tinkering with the endpoints. |
Is your feature request related to a problem?
Our endpoints code is (maybe unnecessarily) scattered across files, has multiple levels of inheritance, and is generally complicated to follow/debug for contributors. As we keep adding more and more endpoints, this situation will only get amplified.
We need to refactor our endpoints to have code that is simpler and more maintainable in the long run.
Describe the solution you'd like
The text was updated successfully, but these errors were encountered: