-
-
Notifications
You must be signed in to change notification settings - Fork 266
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
Add ability to clear short URL visits #1148
Comments
A bit of an edge case and not really useful. Can be done at database level if the specific case ever comes to place. |
Hi! I was about to create a new feature request issue but managed to find this one. Users in our organization have come across 2 scenarios where this feature would be wonderful:
For both of these cases, we would like to wipe the Visit Stats clean for the affected shortlinks so we can be assured that the visit stats page reports as much relevant traffic as possible. Our current band-aid solution is to delete the affected shortlinks and recreate them, which takes time. I checked the CLI entry point documentation for any advanced/hidden features and it seems like there are only options to list visits, but not delete them or manipulate them in any way. Sorry to revive this years-old ticket, but would you reconsider implementing this feature? If not, how can I go about doing this at the database level? Thank you for your time! |
Re-opening this. |
Summary
Allow to reset the visits for a short URL, via endpoint or CLI command.
DELETE /short-urls/{shortCode}/visits{?domain}
short-urls:visits-clear
/short-urls:visits-purge
(start with the wordvisits
so that it is listed next toshort-urls:visits
)Display confirmation on CLI command, stating once done, it cannot be undone.
When shlink-web-client implements the endpoint, also add confirmation where the user needs to perform an action to proceed.
Take into consideration API key roles. An API key should not be allowed to purge visits on a short URL that it cannot fetch (it should work out of the box, but it's worth double checking).
The text was updated successfully, but these errors were encountered: