Launchpad itself

Add links to latest full and delta language pack exported

Bug #146178 reported by Carlos Perelló Marín
2
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
High
Adi Roiban
Launchpad itself 10.04

Bug Description

With the move of language pack exports to production servers we added +latest-delta-language-pack and +latest-full-language-pack links to retrieve latest version of language packs exports.

We don't link to them from Launchpad so they are only available to people that knows about it.

We should add them to the +language-packs page.

Since we are here, maybe we can also add a table for listing current and language pack as described in bug 367844 (ie https://code.launchpad.net/~adiroiban/launchpad/bug-561355/+merge/23249), and there is also the idea of using UTC and not user timezone.

See original description

Related branches

lp:~adiroiban/launchpad/bug-146178
Curtis Hovey (community): Approve (ui)
Brad Crittenden (community): Approve (code)
Diff: 469 lines (+204/-71)
5 files modified
lib/lp/registry/interfaces/distroseries.py (+12/-13)
lib/lp/translations/browser/distroseries.py (+40/-4)
lib/lp/translations/stories/distroseries/xx-distroseries-language-packs.txt (+87/-24)
lib/lp/translations/templates/distroseries-language-packs.pt (+59/-22)
lib/lp/translations/templates/distroseries-translations.pt (+6/-8)
Brad Crittenden (community): Approve (code)
Curtis Hovey (community): Approve (ui)
Eleanor Berger (community): Approve (ui*)
Abel Deuring (community): Abstain (code)
Diff: 469 lines (+204/-71)
5 files modified
lib/lp/registry/interfaces/distroseries.py (+12/-13)
lib/lp/translations/browser/distroseries.py (+40/-4)
lib/lp/translations/stories/distroseries/xx-distroseries-language-packs.txt (+87/-24)
lib/lp/translations/templates/distroseries-language-packs.pt (+59/-22)
lib/lp/translations/templates/distroseries-translations.pt (+6/-8)
Carlos Perelló Marín (carlos)
Changed in rosetta:
importance: Undecided → Medium
status: New → Confirmed
Revision history for this message
Данило Шеган (danilo) wrote :

And now that we link to latest ones directly from DistroSeries:+translations page, that's where we should link to these (i.e. on https://translations.edge.launchpad.net/ubuntu/karmic)

Changed in rosetta:
importance: Medium → High
tags: added: ui
Revision history for this message
Adi Roiban (adiroiban) wrote :

If I understood this bug correctly, the solution for this bug should remove "hardcoded" links like
"http://launchpadlibrarian.net/38015729/ubuntu-hardy-translations.tar.gz" with "+latest-full-language-pack".

This should be done on DistroSeries:+translations and on DistroSeries:+language-packs.

Changed in rosetta:
assignee: nobody → Adi Roiban (adiroiban)
Revision history for this message
Adi Roiban (adiroiban) wrote :

It looks like we have both current and latest language packs.

For a pre-implementation chat I have added them on the DistroSeries:+translations page.

I used menu:translations TALES expression to simplify the template.

Adi Roiban (adiroiban)
description: updated
description: updated
Changed in rosetta:
status: Triaged → In Progress
Revision history for this message
Данило Шеган (danilo) wrote :

I don't think these should be as widely exposed, so they probably shouldn't even appear on +translations page: I probably suggested it when I thought they were about links for "current" language packs. So, they should only be linked from the +language-packs page, imo. I am not so sure about your "simplification" either (Python code is much more complex instead).

Also, if they are one and the same (i.e. current and latest) we shouldn't show them both imo (or, we'd say: "no update").

Revision history for this message
Adi Roiban (adiroiban) wrote :
Revision history for this message
Adi Roiban (adiroiban) wrote :
Revision history for this message
Ursula Junque (ursinha) wrote : Bug fixed by a commit

Fixed in stable r10821 <http://bazaar.launchpad.net/~launchpad-pqm/launchpad/stable/revision/10821>

Changed in rosetta:
milestone: none → 10.04
status: In Progress → Fix Committed
tags: added: qa-needstesting
Revision history for this message
Данило Шеган (danilo) wrote :

I tested this and it looks good, though it'd probably be better if "Language pack being tested" was under the "Active language packs" section now.

tags: added: qa-ok
removed: qa-needstesting
Данило Шеган (danilo)
Changed in rosetta:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.