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
(8.5.4) GH-2153: Future-proof metrics uninstall URL #615
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Quick version bump on the helper JSDoc
Also, while we're here, might be nice to move the definition of METRICS_URL_SET
out of setUninstallUrl
. No reason to re-initialize that constant every time.
src/classes/Metrics.js
Outdated
* | ||
* @private | ||
* | ||
* @since 8.5.3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My bad, let's bump this to 8.5.4
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Latest changes looks good
* Use only crucial params for uninstall type * Move some query string construction to helper * Group METRICS_URL_SET with other consts * Syntax fix Co-authored-by: Ilya Zarembsky <ilya.zarembsky@gmail.com> Co-authored-by: Christopher Tino <4699516+christophertino@users.noreply.github.com>
* Use only crucial params for uninstall type * Move some query string construction to helper * Group METRICS_URL_SET with other consts * Syntax fix Co-authored-by: Ilya Zarembsky <ilya.zarembsky@gmail.com> Co-authored-by: Christopher Tino <4699516+christophertino@users.noreply.github.com>
Shortens the query string for the uninstall URL by including only the ping params we've deemed as crucial.
I've also added a helper method for constructing the query string key value pairs, which hopefully makes it a little easier to read/identify the queries and values getting added to
metrics_url
, and helps prevent potential errors when adding new params.