Mathieu Pillard

Results 203 comments of Mathieu Pillard
trafficstars

Were you logged in with an account that is an author of the extension you received an email for ?

nvm, I see the problem, it's the "." at the end. We should add a space to prevent this from happening since this is a plain text email... ~~Or maybe...

We decided to: - Add a space or remove the period after the links - Add a sentence (like suggested in https://github.com/mozilla/addons/issues/15008#issuecomment-2340081247) saying "This link is accessible only when logged...

https://github.com/mozilla/addons/issues/15072 will add a couple of those reasons, but there are more that are grouped together currently. Essentially this is about expanding `needs_human_review_other` in `Version.get_due_date_reason_q_objects()` to each individual possible NHR,...

As noted in https://github.com/mozilla/addons-server/pull/22900#issuecomment-2506039796 : ideally the form would not use `VIEW_QUEUE_FLAGS`. That would allow us to stop including `needs_human_review_auto_approval_disabled` and `needs_human_review_promoted` in it, which are already displayed through other...

We could consider adding select all/none options as part of this to help with the increase in number of filters.

On top of the fact that there should now be a filter checkbox for each possible reason for something needing human review (and therefore be in the queue), I've updated...

The difference between the 2 promoted ones: - "Was added to a promoted group" is for when the add-on is first added to a promoted group, but wasn't originally part...

> For themes, I tried for stage new versions on By Firefox themes. This theme is in awaiting review in themes queue with By Firefox badge. With " Belongs to...

Bit of an edge case but we should fix it.