Publish on AMO

[ggrossetie]

No description provided.

[ggrossetie]

Rejected again yesterday because the code is "obfuscated":

Your add-on, Asciidoctor.js Live Preview 0.1.4, has been reviewed by an editor and did not meet the criteria for being hosted in our gallery.

Reviewer:
Kris Maglione

Comments:
This version contains binary, obfuscated or minified code. We need to review all of your source code in order to approve it. Please send the following items to [email protected]:

• A link to your add-on listing.
• Whether you're nominating this version for Full Review or Preliminary Review.
• The source files of your add-on, either as an attachment or a link to a downloadable package. Sources archives may be in any of the following formats: zip, 7z, tar, tar.gz, tar.bz2, or tar.xz. They man not be provided using proprietary archive formats, including RAR and DMG.
• For updates, including the source code of the previously approved version or a diff is also helpful.

We will re-enable your rejected version once we have reviewed the code. You can read our policies regarding source code handling here: https://addons.mozilla.org/en-US/developers/docs/policies/reviews#section-binary.

[mojavelinux]

We can put a non-minified version of asciidoctor.js and opal.js into the
extension. Since it's local, there's really no need to minify anyway.

When we cut a 1.5.0 preview version (hopefully in a few days) we'll push
the raw version into the FF add-on.

[ggrossetie]

The code is not minified but I think that the AMO don't like "generated"
Javascript.
I replied with an explanation on how things are built.
Le 27 nov. 2013 00:33, "Dan Allen" [email protected] a écrit :

We can put a non-minified version of asciidoctor.js and opal.js into the
extension. Since it's local, there's really no need to minify anyway.

When we cut a 1.5.0 preview version (hopefully in a few days) we'll push
the raw version into the FF add-on.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/9#issuecomment-29346142
.

[mojavelinux]

We'll have to warm them up to it. We are pioneers. Time to break new
ground. As the popularity of Opal grows, there will be others for whom we
will pave the path.

[ggrossetie]

I discussed on IRC this week with an AMO reviewer. Kris Maglione will hopefully be on IRC this Monday so I'll have a chance to explain him all the greatness about Opal and Asciidoctor 😄

[mojavelinux]

Great news! Definitely be sure to have them get in touch with @adambenyon if the conversation seems to be slipping away. It's definitely in Opal's best interest to get this approval.

I also might know some fine folks at Mozilla that I could nag ;)

[mojavelinux]

Good luck!

[ggrossetie]

Didn't go too well... I just have to wait for their decision.

empathy over process

[ggrossetie]

Any news ? 😢

[mojavelinux]

Hey Guillaume!

I haven't forgotten about this, but I haven't had a chance to follow up on
it yet either. I'll move it up in the priority list, see if I can get the
ball rolling by next week.

[akhayyat]

Is this going to happen?

The README says:

Install extension from Firefox Add-ons site or [...]

But it's not available there!

[ggrossetie]

Is this going to happen?
Hopefully... Maybe... I don't know... That's a good question 😄

The AMO is a long process and I'm currently 42/109 in the queue (I submitted the add-on four weeks ago).
The first submission (10 months ago) didn't go well because the Javascript generated by Opal was judged "obfuscated". After discussing and pleading my case on the forum or via email I decided to try again, let's see how it goes...

The add-on is not indexed but you can install it from https://addons.mozilla.org/fr/firefox/addon/asciidoctorjs-live-preview/

You can also a review to help get it approved 👍

[mojavelinux]

Here's the english version of the add-on page:

https://addons.mozilla.org/en-US/firefox/addon/asciidoctorjs-live-preview/

I found that you have to go into the DOM inspector and uncheck the style pointer-events: none on the link so you can click it.

[mojavelinux]

I found that you have to go into the DOM inspector and uncheck the style pointer-events: none on the link so you can click it.

Oh, that might of been because I am using Chrome. Duh.

[ggrossetie]

@mojavelinux Any news ? Still in the queue 32 of 169... maybe we can add a link on asciidoctor.org ?

[mojavelinux]

@Mogztter No news yet. I'll note it down to follow up.

maybe we can add a link on asciidoctor.org ?

Definitely. Send a PR and I'll merge it in.

...which reminds me, we still need to rewrite this page to include a section about the browser plugins: http://asciidoctor.org/docs/editing-asciidoc-with-live-preview/ We could even consider just having a separate guide. Best place to start is an outline.

[ggrossetie]

#AMO review is endless... I published my add-on 2 months, 3 weeks ago!

[mojavelinux]

😖

Btw, still no word from Mozilla mgmt.

(It almost makes me wonder if the community should counter with an alternate market to apply pressure).

[ggrossetie]

Or they could use the community to do the review by voting or reporting
add-ons. There are just not enough volunteers to review all the add-ons.

I don't know how Opera manage to review add-ons in days... I believe they
have full time employees to do the review.
Le 23 nov. 2014 11:17, "Dan Allen" [email protected] a écrit :

[image: 😖]

Btw, still no word from Mozilla mgmt.

(It almost makes me wonder if the community should counter with an
alternate market to apply pressure).

—
Reply to this email directly or view it on GitHub
#9 (comment)
.

[ggrossetie]

@mojavelinux "5 months in the queue" https://forums.mozilla.org/viewtopic.php?f=21&t=22584

[mojavelinux]

When I read that response, I feel a lot of a pain. But at least it ends on a promising note.

Thanks for the feedback. We'll try to get to your add-on soon, so please be patient.

We can be grateful for something, I suppose ;)

[ggrossetie]

And done! 🎉 https://addons.mozilla.org/en-US/firefox/addon/asciidoctorjs-live-preview/

[mojavelinux]

🍻 and lots of them! Congrats!