945858 - Let developers add version notes to their app

Status: VERIFIED FIXED

(Marketplace Graveyard :: Developer Pages, defect, P1)

Description

[Wil Clouser [:clouserw]]

This is the developer page modifications for bug 945852 (look at the mockups on that bug to see what we're talking about).  

This bug is for the developer page changes.  The UX team is thinking this should just be a textarea under "Description" and above "Categories" on the Edit Listing page.

Right now, in the database we have a "version notes" field for add-on versions which I think makes sense to use here.  Some confusion might appear around automatically added new versions (like when a manifest changes).  We could propagate the field to the new versions - this might mean it gets out of date eventually but would mean it wouldn't disappear seemingly sporadically.  We can adjust the upload flow to encourage filling in new version details if appropriate with the upload changes cvan is thinking about for next quarter.  Any thoughts?

This bug is for:

1) Adjust the Edit Listing page so developers can add notes
2) Expose the new field on the detail page of the API.  No need to increment the API.
3) Add the new field to the API documentation

Comment 1

[Christopher Van Wiemeersch [:cvan]]

We already have Version Notes for each individual packaged app: http://i.imgur.com/GVDpNvc.png

Are we talking about this for hosted apps?

Comment 2

[Wil Clouser [:clouserw]]

(In reply to Christopher Van Wiemeersch [:cvan] from comment #1)
> We already have Version Notes for each individual packaged app:
> http://i.imgur.com/GVDpNvc.png
> 
> Are we talking about this for hosted apps?

Oh, perfect.  I guess I don't have any packaged apps.  If we're already exposing this in the API we can get started on the frontend bug.

We should have the same thing for hosted apps although if the UI is going to get complicated we can ask for help.

Comment 3

[Mathieu Pillard [:mat]]

https://github.com/mozilla/zamboni/pull/1492

Comment 4

[Matt Basta [:basta]]

Didn't we remove this because UX told us to get rid of it?

Comment 5

[Mathieu Pillard [:mat]]

Fixed in https://github.com/mozilla/zamboni/commit/5b145d790345c9b662fbe24a3bb36a12afe373bc

STR:
- Upload an hosted app
- Verify that the edit page contains the release notes field
- Try editing both release notes and another field, it should work :)

- Upload a packaged app
- Verify that the edit page doesn't contain the release notes field
- Try editing basic details, it should work :)

Note: the release notes aren't currently displayed on the consumer pages, that's a separate bug.

Comment 6

[Victor Carciu]

Verified as fixed : http://screencast.com/t/kZORaATLCjl - for hosted app
                    http://screencast.com/t/rucgmpOlIXe6 - for packaged apps