Skip to content
This repository has been archived by the owner on Nov 13, 2024. It is now read-only.

Verify/event data #6

Closed
wants to merge 3 commits into from
Closed

Verify/event data #6

wants to merge 3 commits into from

Conversation

n3f
Copy link
Collaborator

@n3f n3f commented Oct 22, 2024

Fixes https://github.com/Automattic/gold/issues/525

Changes proposed in this Pull Request

  • Adds Sift API object
  • Adds tests to check objects

Testing instructions

  • Setup with wp-env (e.g. npm i && composer install && npx wp-env start)
  • npm run

@n3f n3f changed the base branch from trunk to test/events October 22, 2024 18:17
@n3f n3f self-assigned this Oct 22, 2024
Copy link
Collaborator

@millerf millerf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Base automatically changed from test/events to trunk October 23, 2024 16:02
@n3f
Copy link
Collaborator Author

n3f commented Oct 24, 2024

Closing in favor of #9

@n3f n3f closed this Oct 24, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants