640195 - Remove old Kashubian spellchecker from Dictionaries page

Status: RESOLVED WORKSFORME

(addons.mozilla.org Graveyard :: Administration, defect, P3)

Description

[Jorge Villalobos [:jorgev] (he/him)]

There are currently 2 Kashubian Polish spellcheckers on AMO. The old one (https://addons.mozilla.org/en-US/firefox/addon/kashubian-spell-checker-poland/) is outdated in terms of words it supports and also Firefox compatibility. I'm OK having both of these add-ons on the site, but it would be good that only the better one appears on the Dictionaries page (https://addons.mozilla.org/en-US/firefox/language-tools/).

It used to be the case that this could be done on the Edit pages if you were an admin, but that feature hasn't been ported yet, so I need somebody to make the change directly on the DB.

Comment 1

[Wil Clouser [:clouserw]]

The dictionary page is meant to hold all dictionaries on the site (including multiple options for the same language), we just never got around to automating it.

We can still remove that dictionary, but it might reappear if we get to that point some day.

Comment 2

[Jorge Villalobos [:jorgev] (he/him)]

For now I think it's OK to remove it.

Comment 3

[Wil Clouser [:clouserw]]

In a roundabout way, it sounds like this bug is asking to remove this from the dictionary page: https://addons.mozilla.org/en-US/firefox/addon/kashubian-spell-checker-pl/

I hope that's right.


mysql> update addons set target_locale=NULL, locale_disambiguation=NULL where id=251556 limit 1;
Query OK, 1 row affected (0.82 sec)
Rows matched: 1  Changed: 1  Warnings: 0

Comment 4

[Jorge Villalobos [:jorgev] (he/him)]

No, it's the opposite.

This one should be in the dictionaries page:
https://addons.mozilla.org/en-US/firefox/addon/kashubian-spell-checker-pl/

This one should be removed:
https://addons.mozilla.org/en-US/firefox/addon/kashubian-spell-checker-poland/

Comment 5

[Wil Clouser [:clouserw]]

mysql> update addons set target_locale="csb" where id=251556 limit 1;
Query OK, 1 row affected (0.17 sec)
Rows matched: 1  Changed: 1  Warnings: 0

Comment 6

[Jorge Villalobos [:jorgev] (he/him)]

That reversed the update on Comment 3, but hasn't removed the other one.

Comment 7

[Wil Clouser [:clouserw]]

I looked at the other one, and it shouldn't be in the list at all.  It doesn't have a target_locale afaict.  If it doesn't disappear from the page, I'll look at it later

Comment 8

[Jurk]

I have deleted my version of the add-on. Everything looks good now -- there is just a single version of the add-on (authors have joined their efforts).

Please, close the bug.