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

Cannot remove all recommended themes using django admin #6810

Closed
ioanarusiczki opened this issue Jul 12, 2019 · 5 comments
Closed

Cannot remove all recommended themes using django admin #6810

ioanarusiczki opened this issue Jul 12, 2019 · 5 comments
Labels
migration:no-jira migration:2024 needs:info repository:addons-server Issue relating to addons-server state:stale Issues marked as stale. These can be re-opened should there be plans to fix them.

Comments

@ioanarusiczki
Copy link

Followup for #6312
STR:

  1. Load https://addons-internal.dev.mozaws.net/en-US/admin/models/bandwagon/collection/2143965/change/
  2. Remove all themes from this collection and save changes
  3. Load AMO dev

Expected result:
Recommended themes shelf is no longer available on the hompeage.

Actual result:
Shelf is displayed with 3 themes and See more recommended themes can be accessed
https://addons-dev.allizom.org/en-US/firefox/search/?recommended=true&sort=random&type=statictheme

Notes:
Reproducible on dev with FF68(Win10).

This is the frontend
frontend for themes

The collection is empty in admin
no themes in admin

@eviljeff
Copy link
Member

eviljeff commented Jul 12, 2019

@ioanarusiczki It's just if you remove all of them, right? Removing one or more but leaving at least one works correctly?

@ioanarusiczki
Copy link
Author

@eviljeff Sorry for the late answer, I've been away.

I checked this again with FF68 (win10) and I could reproduce it on dev : adding and removing works but when all themes are removed and django shows no content in this collection the frontend will continue to display the following 3 themes:

504236 Underwater Dolphins
616881 Bubble mode 7
498533 Flag of Colombia

If I leave one the frontend will display 4 themes (what's left in django plus those 3 themes).

But I could not reproduce it on AMO stage. If I remove all themes the frontend shows it correctly.

stage
stage

dev
dev

@eviljeff
Copy link
Member

if the themes are still showing as recommended in the frontend (and they are in the api response too), but aren't in the featured themes collection, then it's possible (probable, imo) they've been directly added as a recommendable add-ons and then approved - i.e. the have DiscoveryItem entries that have been marked as recommendable and then the next version approved)

@ioanarusiczki
Copy link
Author

@eviljeff I managed to delete successfully 498533 Flag of Colombia. It was added like you said, in the Discovery items entries (like the other two mentioned above). In it's place I added with the theme collection 497656 Colombia.

But when I tried to do the same thing for the other two themes - delete them from the Discovery items and add them back using the collection, it did not work anymore.
These two themes:
504236 Underwater Dolphins
616881 Bubble mode 7
still show up in the search API as being recommended. (yet, they are deleted from discovery items).

https://addons-dev.allizom.org/api/v4/addons/search/?recommended=true&type=statictheme

Yet, the badge is no longer there on the detail page. Also when I checked the detail API for these two themes, I see is_recommended:false.

https://addons-dev.allizom.org/api/v4/addons/addon/504236
https://addons-dev.allizom.org/api/v4/addons/addon/616881

@stale
Copy link

stale bot commented Mar 3, 2020

This issue has been automatically marked as stale because it has not had recent activity. If you think this bug should stay open, please comment on the issue with further details. Thank you for your contributions.

@stale stale bot added the state:stale Issues marked as stale. These can be re-opened should there be plans to fix them. label Mar 3, 2020
@stale stale bot closed this as completed Mar 17, 2020
@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
Labels
migration:no-jira migration:2024 needs:info repository:addons-server Issue relating to addons-server state:stale Issues marked as stale. These can be re-opened should there be plans to fix them.
Projects
None yet
Development

No branches or pull requests

5 participants