Skip to content
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

Have feature themes collection membership be treated as recommended #6669

Closed
eviljeff opened this issue May 15, 2019 · 2 comments · Fixed by mozilla/addons-server#11435
Closed

Comments

@eviljeff
Copy link
Member

(At least at first) themes aren't going to go through the same process as extensions to get recommended - they don't have code so don't need a code review so we'll reuse the existing way featured themes are selected (by membership of special collection)

@ioanarusiczki
Copy link

ioanarusiczki commented May 21, 2019

@eviljeff - I tested on AMO dev - FF66(Win10) and here are my results with the associated questions:

so we'll reuse the existing way featured themes are selected (by membership of special collection)

Which means that once the theme is added in the featured collection it's also considered as being recommended and nothing else needs to be done such as adding the recommendable flag from Discovery items and for it's last version (which can be done but if the theme is not part of the featured collection it won't be a real use case). Did I get it right?
Also, I can't tell if that Recommended flag is going away from here, that's if the featured is entirely removed and replaced with recommended.

featured themes from homepage

@ioanarusiczki
Copy link

#13197 has landed so
Static themes which are added to the Featured Themes collection will automatically become Recommended (is_recommended": true) and they'll display the badge.

@KevinMind KevinMind transferred this issue from mozilla/addons-server May 4, 2024
@KevinMind KevinMind added repository:addons-server Issue relating to addons-server migration:2024 labels May 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

3 participants