-
Notifications
You must be signed in to change notification settings - Fork 0
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
Install link of RankMath must change based on the plugin of origin #17
Comments
Scope a solution ✅In
Update tests. Estimate the effort ✅[XS] Would this be limited to just Imagify & WP Rocket? @MathieuLamiot |
@jeawhanlee Thanks!
Those are the only use-cases we have currently, but this could be extended and there are no specific reasons to restrict. Ideally, plugin-family would just add the plugin slug in the link |
LGTM |
@MathieuLamiot Do we need to trigger download, install and activate using the custom url with one-click? |
Yes, the behavior of the button must be completely the same as currently, just using a different place to download the plugin. |
Ok Thanks, the grooming above does not satisfy this, we'll need to update it. |
Moving this back to Grooming in Progress then. Can you look into it? Thanks 🙏 |
Update from Mladen at RankMath: the links will be The links are already working. |
Grooming has been updated to fit AC |
LGTM |
@jeawhanlee @piotrbak It woudl be great if this update is:
I guess we can just avoid updating the plugin-family version in Imagify for now. Thanks 🙏 |
Context
See #16
The validation of the end-to-end behavior depends on https://gitlab.group.one/rankmath/rankmath.com-website/-/issues/5
Currently, the "INSTALL" button for RankMath downloads & installs the plugin from the WP public repo.
Expected behavior
The Install button for RankMath must trigger the download & install of RankMath plugin from the following link:
rankmath.com/downloads/plugin-family/_plugin_slug_
where_plugin_slug_
should be replaced by the plugin name of the plugin from which the advertisement is displayed:imagify
for Imagifywp-rocket
for WP RocketAcceptance Criteria
Additional information
You can contact Uzair or Mladen from RankMath to access the download counter.
See more details on group.one Slack #saas-eng-plugin-family
Once this is implemented in plugin-family, both Imagify & WP Rocket must be updated accordingly.
The text was updated successfully, but these errors were encountered: