-
Notifications
You must be signed in to change notification settings - Fork 82
Feature Request: Add setting to exclude posts that have been on the most-viewed list for a certain duration #394
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
Comments
Hi @grantbarrett, Thanks for the suggestion! I got one better (?), how about for example adding new time range options, like:
That way your popular posts list will only pull data from eg. the current week, completely ignoring past data. That way, once the selected "current X" time range ends the next one will automatically only list items that are being viewed / commented on within the new time range. With this, you won't need to manually exclude IDs each week. ...hope I managed to explain myself, it makes sense in my head but it's Sunday night here and my brain is not in the freshest state right now haha 😅 |
That would be less processor-intensive, for sure! I think that would be an elegant solution to my situation. |
Alright, I may create a separate branch to implement this functionality and ask you to test it out for me if that's OK. I'll let you know, no ETA though but hopefully soon. |
Alright, took a little while but I just pushed a new branch called this-x-time-ranges that implements new time ranges as stated on this comment. Branch: https://github.com/cabrerahector/wordpress-popular-posts/tree/feature/this-x-time-ranges Please give them a try and let me know if you have any comments/questions @grantbarrett |
I will check it out! |
Can you give guidance on how to test it with the shortcode? That's how I implement it. Thanks! @cabrerahector |
Ah, forgot to mention that haha @grantbarrett With this change, the For example:
As to how to get the actual plugin with these changes:
Once you're done testing:
|
Thanks! That displays posts! It is acting counter to expectations, however. I am using Also, can we get a |
Yes, that shortcode will display the most viewed posts of this month regardless of their publication date. If you want it to only list the most viewed posts that were also published within the current month use the
|
Oh, see. I guess I kind of lost sight of what we were doing here. Sorry! I will look into freshness. |
Don't worry about it, it's fine. I'll look into adding a About some settings not being exposed on the Stats dashboard, someone already made a request to include them in #273 - I just keep forgetting about it (ᵕ・ᴗ・) |
Is your feature request related to a problem? Please describe.
Would it be possible to add a setting to allow exclusion of posts that have appeared in the top lists for a certain amount of time?
Describe the solution you'd like
Some kind of option in the shortcode to use the WPP logs to track if a post has been in the top lists in a way that meets the plugin's current criteria and then give the user the option of not displaying it. For example, if I have a new "items of the week" article every week I might want to make them not display in the top posts lists after seven days but also not want to have to keep editing my shortcode to add more article IDs to exclude every week.
If post meets current settings X and post has met those criteria for period of time Z then don't display post for period of time Q.
The text was updated successfully, but these errors were encountered: