-
Notifications
You must be signed in to change notification settings - Fork 2
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
Already published post gets counted as data-point #1
Comments
Validation of this issue from @dreeves in the Beeminder forum:
This can be easily addressed with a more selective choice of WP action hooks when updating posts. |
The issue is our use of the (From the function's documentation: "When a post is saved, the post status is "transitioned" from one status to another, though this does not always mean the status has actually changed before and after the save". Misleading language. The Thus, code hooked to |
To inform a better choice of hooks to act on, I've drafted the user stories below to document expected behavior. Even though this is dirt simple, I'd appreciate a sanity check before moving forward with the fix. (ATTN @dreeves, @Sodaware) Bloggerbee user definition: Someone with Stories: As a Bloggerbee…
Translation to technical requirements/rules:
|
If you edit posts which have been published before installing the plugin another data-point will be pushed to the beeminder goal.
I think the problem lies here: https://github.com/beeminder/beeminder-ping/blob/master/beeminder-ping.php#L107
The text was updated successfully, but these errors were encountered: