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

feedback from review to get final approval #29

Open
mitechie opened this issue Dec 22, 2014 · 0 comments
Open

feedback from review to get final approval #29

mitechie opened this issue Dec 22, 2014 · 0 comments

Comments

@mitechie
Copy link
Member

When submitting an update we get review feedback on things that should be done to become a completely approved and searchable extension:


Comments:
Your preliminary review request has been approved.

Please note the following for the next update:

  1. Your add-on prints debugging information to the Error Console, which is generally not allowed in production add-ons.

  2. Due to both security and performance reasons, innerHTML (or similar method) should not be set using dynamic values. This can lead to security issues or fairly serious performance degradation.

For inserting text, textContent or createTextNode() should be used instead of innerHTML.
For inserting HTML, the safer method is to use createElement(), textContent, appendChild() instead of innerHTML.
Removing the innerHTML will improve add-on's prospect of approval.

Thank you.

Your add-on will now appear in search results and categories with some limitations. After 10 days you may request full review to remove these limitations and enable additional features. To learn more about the review process, please visit https://addons.mozilla.org/developers/docs/policies/reviews#selection

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

No branches or pull requests

1 participant