Skip to content

Loading…

Add to addons.mozilla.org #550

Closed
sgarrity opened this Issue · 47 comments
@sgarrity

Would be nice to be able to find/install via addons.mozilla.org. Thanks.

@Deathamns

It will be there when it's ready. But it's still in beta.

To make it happen, at least these issues should be addressed:
Preliminary Review

  • Support only from version 29 (for the time being).
  • #507
  • #522
  • #553

Full Review

@nkestrel

AMO's preliminary review process is intended for experimental addons like this.

@pwr22

It might be good to take advantage of this, because then people testing these (beta) builds won't have to manually come check for updates

@Deathamns

@nkestrel If it will be sent to AMO, no one will consider it as beta.

@pwr22 They don't have to install it manually, but first they will have to wait a bunch of time before it will be reviewed. The first review usually takes weeks, then if they don't like something, it will be rejected, and after we send a fixed version weeks of waiting follows again, because it will be put at the queue again. So, while they would review it, we can finish it, and send a final version instead of a beta.

@nkestrel

no one will consider it as beta

"ranked lower than fully-reviewed add-ons. Install buttons will have caution stripes and a notice that the add-on is experimental and not fully reviewed by Mozilla"

after we send a fixed version weeks of waiting follows again

"Preliminary reviews are appropriate for experimental add-ons and provide a way to get user testing and feedback without going through the longer, more thorough review process. We aim to complete these reviews in under 3 days."

@Deathamns

It doesn't work like that. Users expect quality from things that are on AMO.

Also, what you're quoting maybe was true a few years ago, but nowadays reviews are taking quite some time. Here's the latest report.

Most nominations for full review are taking less than 9 weeks to review.
Most preliminary reviews are being reviewed within 6 weeks.

@gorhill

I remember this article from not too long ago:

each review cycle takes weeks to complete. Couple a capricious rejection process with a latency that’s measured in weeks, and you’ve got the primary reason that we’ve released five full updates to our Chrome extension, in the time that we’re still awaiting one full approval from Mozilla

@nkestrel

In my recent experience, the first preliminary review for an addon can take weeks like a full review, but after that it is possible for subsequent preliminary reviews to be granted in a matter of minutes.

@Deathamns

@gorhill That article isn't far from the reality, but it depends on many factors...

@nkestrel Your extension seems to be relatively simple, so it's easier to review.

Giving it a second thought, exactly because of these long delays we probably should send it for a preliminary review, so basically while they get to reviewing it, we may finish it in that time.

@Deathamns

@gorhill I marked 507 for Full Review. So, what then? May I send it or you will?

@gorhill

If you don't mind taking care of the AMO version, that would be great -- I can't take any more tasks, rather the opposite.

Given how the project has grown, I will open an issue "Change official ownership to 'The µBlock Development Team'" in code/docs, though in the store it's fine to just use our own name for publication (gorhill = Chrome/Opera store, Deathamns = Firefox add-ons repo, Chris Aljoudi = Safari, etc.)

@gorhill

Was it submitted to AMO?

@Deathamns

No. I'm waiting for 0.8.6.0.

@Deathamns

Could we include the non-minified versions of the JS files in the lib directory (since reviewers may have problem with that)?
Also Punycode.js has newer versions.

@chrisaljoudi

@gorhill I think @Deathamns meant to merely include the non-minified versions, not necessarily use those (so just having publicsuffixlist.js in addition to publicsuffixlist.min.js in lib). I think 084f092 replaces them fully.

There are often certain, though minimal, performance advantages to using minified versions of libraries.

@Deathamns could you confirm?

@gorhill

performance advantages to using minified versions of libraries

To reduce bandwidth and possible load/parse time. I doubt it matters for a long running background page

@chrisaljoudi

@gorhill yep, indeed. I guess it is insignificant; apologies.

@Deathamns

@chrisaljoudi The performance gain from parsing is probably close to irrelevant for these small files, but the point here is to help the work of reviewers. If we would still include the minified versions, they would still need to do the extra work for verifying if the files are the same (by the way, I tried minifying publicsuffixlist.js from GitHub with the tool @gorhill mentioned in that file, but the minified results weren't the same, which would get a guaranteed rejection, since they won't start hunting for the correct file on GitHub, or tweak on the settings in the minifyer).

@chrisaljoudi

@Deathamns sure, that makes sense.

It was quite silly to think about performance differences; sorry about that.

@Deathamns

@gorhill Have you tried uploading any previous version to AMO? Because I get the error: Duplicate UUID found.
If it was you, then you can finish publishing it, or delete it to allow me to send it.
If not you, then someone else was kind enough to upload it for us (this happened with my extension too about a year ago).

@gorhill

If not you, then someone else was kind enough to upload it for us

What now? (I guess you were being sarcastic with "kind enough"). Who knows, maybe someone not nice uploaded it after putting in not nice stuff in it?

@Deathamns

I'll write an e-mail then, but first I wanted to know if it was you, so we could resolve the issue faster without contacting AMO people. But if not, then I'm sending a mail...

@gorhill

I wanted to know if it was you

I did try once just to see what would happen but it also got rejected because of "Duplicate UUID", which led me to think you started the process -- which I thought was nice that this was all taken care of.

@nkestrel

Just change the <id> value in install.rdf. Preferably don't use a generated UUID as it is unnecessary and unfriendly (the id is used for the xpi filename). Most addons use the structure name@website or name@author, so something like ublock@gorhill.

@gorhill

@nkestrel Would uBlock@https://github.com/gorhill/uBlock works?

@Deathamns

That id is not really seen by users, so its user-friendliness doesn't really matter.
But the real problem is that quite a few people have already installed it with the current id, if I would change the id for the AMO version, then no one would get automatic updates from AMO.

@sn260591

@Deathamns But in what a difference if at the moment people all the same update ublock manually? You can change the id in the next version (0.8.6.1) and put it on the AMO.

@pwr22

@sn260591 I believe all those existing installations will begin to update automatically

@ghost

Any update on the AMO front?

@Deathamns

I just received a response from the editors, hopefully I can upload it soon.

Note that I'll send version 0.8.6.0 with a small change: it will be installable only for Firefox 29+.

@gorhill What kind of screen-shots you want to have on AMO?

@gorhill

@Deathamns re. screenshots, whatever you think a user may want to see before installing the add-on.

Edit: Not the charts though, as these were made using data from Chromium, so we can't use them for Firefox.

@gorhill

Actually, I will repeat this manual benchmark for Firefox.

@matrixik

Looks really nice. Maybe in next benchmark you can also check μ Adblock for Firefox: https://addons.mozilla.org/en-US/firefox/addon/micro-adblock/
I'm using it now and it feel quite lightweight and have some nice additions like: Remove Google/Yahoo/Naver/Daum redirect URLs from search page. and Option to enable/disable Mozilla Polaris tracking protection.

@gorhill

@matrixik The details of the benchmark are here. You just have to follow carefully and exactly every step to find out.

@pwr22

@gorhill thanks for including Bluhell, barely below uBlock :)

@gorhill

@pwr22 I wondered if uBlock could even be below Bluhell if cosmetic filters were not loaded (there is an option for this in uBlock), which would make uBlock and Bluehell a bit more comparable -- as Bluhell doesn't have DOM filtering.

@pwr22

I'm very happy with uBlock now and will be switching. It is the memory * tab and more performance of ABP that caused me to switch in the first place. I have a low power netbook and ABP was actually locking up FF on pageloads

@Xylemon

"Implement toolbar button and pop-up with native API", is there any progress on this?

@eridal

I just installed uBlock on my desktop, and removed the ABP .. now after firefox have sync'ed across all my devices I have no adblocker except on desktop

I would need to reinstall ABP, until this gets out there

@ghost Unknown referenced this issue
Open

[Firefox] Extension signing #764

@ghost

Make uBlock available for Pale Moon and fantasies such as Mozilla's latest won't get into the way.
I'm running Firefox but should Mozilla continue their dictatorship (see what happened with Australis) I'd be seriously thinking of abandoning the ship. Whatever, should this extension signing prevent uBlock in any way of being available on Firefox that I'd quit this browser. I've been waiting too long for an add-on that would perform as uBlock does, so if it had to be blocked by a company's hysteria I'd move away from that company, and that is no joke.

@sgarrity

Awesome, thanks.

@sgarrity sgarrity closed this
@coctic

Will new versions be on addons.mozilla.org? Now it says the current verison is 0.8.8.2, Version 0.9.1.0 can be found deep in versions history.

What about newer versions from github?

@EchoDev

@coctic
New version have to be reviewed by Mozilla before they get an 'official' update on the addons.mozilla.org site but you are free to just update using the xpi provided on github.

@coctic

I know, and I updated manually. I just wanted to ask about plans to upload new versions to AMO. If the problem is with review, then I'll just wait.

Thanks.

@gorhill gorhill locked and limited conversation to collaborators
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Something went wrong with that request. Please try again.