Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Hide unreviewed versions (if there is at least one reviewed version) on the 'all versions' page #3704

Closed
kuremu opened this issue Oct 10, 2016 · 9 comments
Assignees

Comments

@kuremu
Copy link

kuremu commented Oct 10, 2016

Describe the problem and steps to reproduce it:

Vimperator version (3.15.0) is linking to a missing page.

To reproduce go to https://addons.mozilla.org/en-US/firefox/addon/vimperator/versions/ and click on "Continue to Download" for version 3.15.0

What happened?

404

What did you expect to happen?

A download link for the new version

Anything else we should know?

@wagnerand
Copy link
Member

wagnerand commented Oct 10, 2016

This version has not been approved yet. The download button is different, but we should probably add additional info or hide unreviewed versions from that page.

@jvillalobos Any objections on hiding unreviewed versions (if there is at least one reviewed version) on the 'all versions' page?

@EnTeQuAk
Copy link
Contributor

EnTeQuAk commented Oct 10, 2016

Hmm, curious if this is a duplicate of #3639?

@jvillalobos
Copy link
Contributor

I'm in favor of hiding them. It's consistent with what we want to do in #3348.

@kuremu
Copy link
Author

kuremu commented Oct 11, 2016

@wagnerand I'm sure I have downloaded unreviewed versions like this in the past, hence the confusion.

@EnTeQuAk EnTeQuAk changed the title Vimperator version 3.15.0 link leads to 404 Hide unreviewed versions (if there is at least one reviewed version) on the 'all versions' page Oct 28, 2016
@diox diox self-assigned this Jan 3, 2017
@diox diox closed this as completed in #4205 Jan 4, 2017
@ValentinaPC
Copy link

Verified on AMO-dev FF50.1(Win 7)

  • Unreviewed versions are not listed in all versions page
    allversions
    unreviewed

  • Beta version (from beta channel)- automatically reviewed and signed - is listed in all versions development channel page under the text "Caution: Development versions of this add-on have not been reviewed by Mozilla."
    dev

@diox : From what you said yesterday on irc I understood that beta versions are also not listed in public pages - is this correct?
And second - by who are the beta version reviewed, if not "by Mozilla"?
Thanks!

@diox
Copy link
Member

diox commented Jan 5, 2017

correct, in a way : beta versions are not listed in the "regular" version list, they are listed on the development channel versions list instead.

beta versions are not reviewed, they are automatically signed.

@ValentinaPC
Copy link

Based on the previous 2 comments I'll mark this as fixed.

@jvillalobos
Copy link
Contributor

Actually, we do want to show beta versions on the All Versions page. There's a link from listing pages (in the Developer Channel section) pointing to the All Versions page so you can see previous beta versions. I don't know if this should be reopened or a new issue should be filed, since it's already live.

@jvillalobos
Copy link
Contributor

Never mind. I just realized that, in this particular case I was looking at, there are two links pointing to the different version list pages.

dev-channel

The compatibility warning box has a View other versions link that points to the normal All Versions page. I suppose we can live with that situation for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants