Validation Page has some old links and expired advice

RESOLVED FIXED in 2012-10-18

Status

addons.mozilla.org Graveyard
Add-on Validation
--
minor
RESOLVED FIXED
5 years ago
2 years ago

People

(Reporter: Gregg Lind (Fx Strategy and Insights - Shield - Heartbeat ), Unassigned)

Tracking

unspecified
2012-10-18

Details

(Whiteboard: [qa-])

Attachments

(1 attachment)

(putting on my UX hat)

Looking at:

https://addons.mozilla.org/en-US/developers/addon/test-pilot/validation-result/131205

1) """Error: The validation process took too long to complete. Contact an addons.mozilla.org editor for more information."""

I have no idea of how to contact such an editor.  Maybe link somewhere better? Or just say irc.m.o #addons ?


2)  https://developer.mozilla.org/en/Firefox_1_for_developers is linked but doesn't exist.  I hope we don't need to stay Fx compatiable

3)  for the 'innerHtml' warnings (`innerHTML should not be set dynamically`) , it would be nice to link to ref for what is safe and unsafe practice there.

4)  """Add-on contains JAR files, no <em:unpack>

Warning: The add-on contains JAR files and does not set <em:unpack> to 'true'. This can result in performance issues in add-ons that target Gecko 4. It is recommended that you consider no longer using JAR files to package your chrome files."""

Is this still true?  Gecko 4?  



This are all *super minor things*

Comment 1

5 years ago
I don't have access to the page that you've provided, but if you can attach the XPI or pastebin the output, I can take a closer look.

(In reply to Gregg Lind (User Research - Test Pilot) from comment #0)
> (putting on my UX hat)
>
> 1) """Error: The validation process took too long to complete. Contact an
> addons.mozilla.org editor for more information."""

This message is a problem that we're dealing with now. The error that causes it should not be happening and we're working to fix the problem. You can track it in bug 785531.

> 2)  https://developer.mozilla.org/en/Firefox_1_for_developers is linked but
> doesn't exist.  I hope we don't need to stay Fx compatiable

I can't find any reference to the above URL in the validator's codebase. Is this a part of the results?

> 4)  """Add-on contains JAR files, no <em:unpack>
> 
> Warning: The add-on contains JAR files and does not set <em:unpack> to
> 'true'. This can result in performance issues in add-ons that target Gecko
> 4. It is recommended that you consider no longer using JAR files to package
> your chrome files."""

This should read "Gecko 4 and above". I'll fix this in an upcoming push.
(In reply to Matt Basta [:basta] from comment #1)
> > 2)  https://developer.mozilla.org/en/Firefox_1_for_developers is linked but
> > doesn't exist.  I hope we don't need to stay Fx compatiable
> 
> I can't find any reference to the above URL in the validator's codebase. Is
> this a part of the results?

Yes, it's part of the results page. I think it broke since the Firefox 10 bump. There must be some 1 digit version assumption somewhere.

Comment 3

5 years ago
Can you take a screenshot for me?
Created attachment 655818 [details]
Screenshot of compat validation showing 'Firefox 1' link
https://github.com/mozilla/zamboni/pull/442

(I gave up on testing it myself after I couldn't get my.conf working, and otherwise get the settings_local.py right.  Anything to make that *even simpler* would have helped.  And yes, I realize it's already very simple!)

Comment 6

5 years ago
Merged:

https://github.com/mozilla/zamboni/commit/2b02c171a5406e7a5e1d6719d6307261ccffb441

Comment 7

5 years ago
Fixed for good:

https://github.com/mozilla/zamboni/commit/3fed25a3fd78a54e5563e54edd1cce9ab1fe3b6c
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2012-10-18
Whiteboard: [qa-]
(Assignee)

Updated

2 years ago
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.