Version uploaded while another version is PUBLIC_WAITING goes to wrong queue

VERIFIED FIXED in 2013-07-18

Status

Marketplace
Reviewer Tools
P3
normal
VERIFIED FIXED
4 years ago
4 years ago

People

(Reporter: robhudson, Assigned: mat)

Tracking

2013-07-18
Points:
---

Details

(Whiteboard: p=2)

(Reporter)

Description

4 years ago
When a new app is uploaded and the app has a prior version that is approved but waiting, the new version should appear in the updated queue.

What happens when the dev has 2 approved but waiting versions is another question. See bug 867570.
(Reporter)

Updated

4 years ago
Priority: -- → P3
Whiteboard: p=2
(Assignee)

Updated

4 years ago
Assignee: nobody → mpillard
(Assignee)

Comment 1

4 years ago
When following those steps, I don't see the app in any queue. At all. 
Here is what I did:
- Uploaded v1
- Reviewed it, but changing the requirements, making the version approved but waiting
- Uploaded v2
- v2 never showed up in any review queue but it's shown as pending approval in the devhub

However, if as a dev I make the app public, it correctly swiches v1 to public and then v2, still pending approval is correctly sent to the reviewers queue.

I can try to fix that behaviour and make sure we always sent in the to update queue, but since you mentioned "wrong queue" I want to make sure what I'm seeing is coherent with what you had. Is it ? :)
Flags: needinfo?(robhudson.mozbugs)
I can reproduce using the steps in comment 1 on -dev.  Attempted both using an app where I chose delayed publishing on submission and for an app where I deselected a device so got delayed publishing just for that version.  In both cases the app showed up in the main submission queue.
(Assignee)

Comment 3

4 years ago
I'll retry and fix.
Flags: needinfo?(robhudson.mozbugs)
https://marketplace-dev.allizom.org/reviewers/apps/review/blesuscri-idotilant-abertir
and
https://marketplace-dev.allizom.org/reviewers/apps/review/mezomcusings-britonzontack-

I note that for both of them the statuses of the APP is pending approval, not approved but waiting.  But after the first review the status is correct.  So I would therefore assume the problem is in the submission code - its resetting the status to pending approval if its approved but waiting.
(Assignee)

Updated

4 years ago
Status: NEW → ASSIGNED
(Assignee)

Comment 5

4 years ago
Fixed in https://github.com/mozilla/zamboni/commit/fcb96aeb62a379a6923c0c00268c0d03087c0217

Steps to reproduce:
- Upload a packaged app
- Review it, but changing the requirements, making the app 'approved but waiting'
- Back in the developer pages, don't make the app public, instead upload a new version
- The app should still be shown as 'approved but is not public' in the developer status page, with one version 'Pending approval' and the other 'Approved but waiting'.
- The app should show in the updates queue in the reviewer tools
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
(Assignee)

Updated

4 years ago
Target Milestone: --- → 2013-07-18

Comment 6

4 years ago
(In reply to Mathieu Pillard [:mat] from comment #5)
> Fixed in
> https://github.com/mozilla/zamboni/commit/
> fcb96aeb62a379a6923c0c00268c0d03087c0217
> 
> Steps to reproduce:
> - Upload a packaged app
> - Review it, but changing the requirements, making the app 'approved but
> waiting'
> - Back in the developer pages, don't make the app public, instead upload a
> new version
> - The app should still be shown as 'approved but is not public' in the
> developer status page, with one version 'Pending approval' and the other
> 'Approved but waiting'.
> - The app should show in the updates queue in the reviewer tools

I followed your steps, the app is correctly displayed with both version in developer status page, but I was not able to verify last step because of bug 894394

Comment 7

4 years ago
Verified as fixed last step too : http://screencast.com/t/XMYTMr8G
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.