-
Notifications
You must be signed in to change notification settings - Fork 5.3k
[Components] wordpress_com #15127
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
[Components] wordpress_com #15127
Comments
Hi @michelle0927 @SokolovskyiK, the ticket was moved to ready for release due to an automation bug on my side. It is actually changes required. Could you check the test report below and create a new PR to fix them? https://vunguyenhung.notion.site/Components-wordpress_com-1e3bf548bb5e815dba87d480e611b16d?pvs=4 |
Hello everyone, I have tested this PR and there're some test cases failed or needed improvement. Please check the test report below for more information |
@vunguyenhung |
Hi @michelle0927 @michelle0927 I tested the sources locally and then via pd dev, but the problem is that I only have 10 tries per day, so it's difficult to test everything thoroughly. Is there some kind of dev mode or something where I can increase the number of tries? =) |
@SokolovskyiK definitely yes — we really appreciate all your contributions! DM me on Slack and we'll get you set up: |
Hi @SokolovskyiK, sorry for merging that first PR too soon. When this one is ready for a code review, would you mind moving it from "Changes Required" to "Ready for PR Review" (top right-hand side of this page)? |
Hi @michelle0927. No problem!
|
@SokolovskyiK Sorry, didn't realize you didn't have permission! Moved it to Ready for PR Review! |
@michelle0927 Great. Thanks |
Hello everyone, I have tested this PR and there're some test cases failed or needed improvement. Please check the test report below for more information |
Hi @vunguyenhung |
Hey @SokolovskyiK, yeah the test report is not updated. You can fix only the comments I made. Please ignore the test report for now. Thank you |
Hello everyone, I have tested this PR and there're some test cases failed or needed improvement. Please check the test report below for more information |
I fixed the subscriber issue. That’s when I realized that during my tests, I created a new WP account and clicked the Subscribe button on the website. And the API only fetches one type at a time (email or wpcom). |
Hi everyone, all test cases are passed! Ready for release! Test report |
wordpress_com
URLs
Polling Sources
new-post
Description
Emit new event when a new post is published on the site. Required props: site ID or URL. Optional props: post type (e.g., blog post, page).
new-comment
Description
Emit new event when a new comment is added to any post. Required props: site ID or URL. Optional props: post ID to filter comments for a specific post.
new-follower
Description
Emit new event when a new follower subscribes to the site's blog. Required props: site ID or URL. No optional props.
Actions
create-post
Description
Create a new post on the site. Required props: site ID or URL, post title, content. Optional props: post status (draft, published), post categories, tags.
upload-media
Description
Upload a media file to the site's library. Required props: site ID or URL, media file (binary or URL). Optional props: media title, description.
delete-post
Description
Delete an existing post from the site. Required props: site ID or URL, post ID. No optional props.
The text was updated successfully, but these errors were encountered: