Skip to content
This repository has been archived by the owner on Nov 27, 2019. It is now read-only.

fix: Avoid JS errors from external file from normandy.cdn.mozilla.org #561

Merged
merged 2 commits into from
Aug 2, 2016

Conversation

freaktechnik
Copy link
Contributor

When running tests I'd see JS errors from a selfrepair.js. This change makes Firefox not download that file.

@freaktechnik
Copy link
Contributor Author

freaktechnik commented Jul 30, 2016

Hmm, "browser.reader.detectedFirstArticle": true would disable the reader view UI tour at the startup. Should I add that to this patch or make a separate pr?

@kumar303
Copy link
Contributor

kumar303 commented Aug 1, 2016

would disable the reader view UI tour

I'd say go ahead and add that pref to this patch

@kumar303
Copy link
Contributor

kumar303 commented Aug 2, 2016

Thanks!

@kumar303 kumar303 merged commit 73c9d41 into mozilla-jetpack:master Aug 2, 2016
@kumar303
Copy link
Contributor

kumar303 commented Aug 2, 2016

@freaktechnik thanks again for this. If you feel like it, you could also port this to web-ext :) mozilla/web-ext#406 but it's not urgent

@asamuzaK
Copy link
Contributor

asamuzaK commented Aug 3, 2016

Looks like Travis is failing... Build #780 - mozilla-jetpack/jpm - Travis CI

@freaktechnik
Copy link
Contributor Author

This is likely due to travis using the release Firefox version and that one no longer working with jpm. (See #468 )

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

Successfully merging this pull request may close these issues.

3 participants