Closed
Bug 1195229
Opened 10 years ago
Closed 10 years ago
[bs][Desktop] Outdates searchplugins: pik.ba redirect to olx.ba, tajpi.ba doesn't exist anymore
Categories
(Mozilla Localizations :: bs / Bosnian, defect)
Mozilla Localizations
bs / Bosnian
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: flod, Assigned: flod)
References
Details
(Keywords: productization)
Attachments
(1 file)
|
15.29 KB,
patch
|
Pike
:
review+
|
Details | Diff | Splinter Review |
http://hg.mozilla.org/releases/l10n/mozilla-aurora/bs/file/33af5e01dcc8/browser/searchplugins/
pik.ba redirects, I have no idea if the old website was the same
tajpi.ba is completely broken and should be removed
Comment 1•10 years ago
|
||
You can freely remove tajpi.ba, and Pik.ba was recently acquired by an international investment group OLX, so they changed the domain to olx.ba, everything else remained the same.
| Assignee | ||
Updated•10 years ago
|
Assignee: nobody → francesco.lodolo
| Assignee | ||
Comment 2•10 years ago
|
||
For reference: searchplugins were set up in bug 590133.
| Assignee | ||
Comment 3•10 years ago
|
||
Updated searchplugin with:
* Updated favicon, .ico with 16+32px taken from their website http://v2-assets.pik.ba/common/dizajn/img/faviconolx.ico (removed 64px from the original file)
* Same Name and Description of the Opensearch plugin exposed on their website (this one is still using the old favicon and domain): http://www.olx.ba/sugestije/firefox
* Updated search URL based on their searchplugin, added suggestions.
Searchplugin is available from testing here.
https://l10n.mozilla-community.org/~flod/testsp/#
Kerim, can you give it a test? Everything seems to work as expected for me.
Flags: needinfo?(kerim)
Comment 4•10 years ago
|
||
Tested and works fine. Please remove Pik.ba since you've added OLX.ba and its new icon.
Flags: needinfo?(kerim)
Comment 5•10 years ago
|
||
And thank you very much for your superb effort. You are a true great, Francesco ;)
| Assignee | ||
Updated•10 years ago
|
Attachment #8662322 -
Flags: review?(l10n)
| Assignee | ||
Comment 6•10 years ago
|
||
(In reply to Kerim Kalamujic from comment #4)
> Tested and works fine. Please remove Pik.ba since you've added OLX.ba and
> its new icon.
Thanks for the feedback Kerim.
The patch already renames pik.xml as olx.xml (basically removing the old pik.ba file) and fixes list.txt, "hg move" just makes more evident that pik.ba has become olx.ba
Updated•10 years ago
|
Attachment #8662322 -
Flags: review?(l10n) → review+
| Assignee | ||
Comment 7•10 years ago
|
||
https://hg.mozilla.org/releases/l10n/mozilla-aurora/bs/rev/659a43c9457b
https://hg.mozilla.org/l10n-central/bs/pushloghtml?changeset=659a43c9457b
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•