Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Interaction: Use Rest Controller structure #1149

Open
wants to merge 4 commits into
base: trunk
Choose a base branch
from
Open

Interaction: Use Rest Controller structure #1149

wants to merge 4 commits into from
+250 −47

Conversation

obenland
Copy link
Member

Follow-up to #1146.

Proposed changes:

  • Updates Interactions endpoint to extend WP_REST_Controller.
  • Adds unit tests for Interactions endpoint.

Other information:

  • Have you written new tests for your changes, if applicable?

@obenland obenland added [Type] Janitorial Skip Changelog Disables the "Changelog Updated" action for PRs where changelog entries are not necessary. labels Jan 10, 2025
@obenland obenland requested a review from a team January 10, 2025 21:18
@obenland obenland self-assigned this Jan 10, 2025
@obenland
Remove schema argument
1bb1772 
This endpoint doesn't have a schema.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pfefferle pfefferle pfefferle left review comments

At least 1 approving review is required to merge this pull request.

Assignees

@obenland obenland

Labels
Skip Changelog Disables the "Changelog Updated" action for PRs where changelog entries are not necessary. [Type] Janitorial
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@obenland @pfefferle