Merge lp://qastaging/~hyuchia/noise/releases-info into lp://qastaging/~elementary-apps/noise/trunk

Proposed by Diego Islas Ocampo
Status: Merged
Approved by: Danielle Foré
Approved revision: 2052
Merged at revision: 2058
Proposed branch: lp://qastaging/~hyuchia/noise/releases-info
Merge into: lp://qastaging/~elementary-apps/noise/trunk
Diff against target: 97 lines (+55/-35)
1 file modified
data/org.pantheon.noise.appdata.xml.in (+55/-35)
To merge this branch: bzr merge lp://qastaging/~hyuchia/noise/releases-info
Reviewer Review Type Date Requested Status
Jeremy Wootten code Approve
Review via email: mp+318864@code.qastaging.launchpad.net

Commit message

Add the release information to the AppData.xml to provide appcenter support

Description of the change

Add the release information to the AppData.xml to provide appcenter support

To post a comment you must log in.
Revision history for this message
Cody Garver (codygarver) wrote :

In order to ease the burden on translators, please use the following common descriptions:

* Security fixes
* Various crash fixes
* New translations
* Minor bug fixes

Please do not use the following:
* Minor bug fixes and translations updates.
* Apply minor bug fixes
* Translations Updates

Also I am not sure about the <p> paragraphs, why are they not all <li> list items?

Revision history for this message
Diego Islas Ocampo (hyuchia) wrote :

The paragraphs are mainly to keep appstream-util happy, but I think they should be added so there is a small description of every release instead of just a list of changes, the description can be used for a preview of the update and the description + list for an expanded view.

I'll change the descriptions for the common ones but should I also limit the file to 4 releases (or maybe just the ones for the current elementary release cycle)? That may also help ease the burden.

Revision history for this message
Cody Garver (codygarver) wrote :

Yeah I think you're right, start at 0.4

2052. By Diego Islas Ocampo

Update descriptions and reduce releases to current cycle

Revision history for this message
Diego Islas Ocampo (hyuchia) wrote :

All right, I've removed the releases from other cycles and used the common descriptions.

Revision history for this message
Jeremy Wootten (jeremywootten) wrote :

Passes appstreamcli validate --pedantic.

review: Approve (code)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches