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

Background.js fix: line 1347 - path fix #51

Closed
YasminRodriguez1 opened this issue Jun 5, 2016 · 3 comments
Closed

Background.js fix: line 1347 - path fix #51

YasminRodriguez1 opened this issue Jun 5, 2016 · 3 comments
Assignees
Labels
bug This is a bug with the extension pr-high This is a high-priority issue. If you want to help out, try one of these if you can!
Milestone

Comments

@YasminRodriguez1
Copy link

in js/background.js:
line 1347: chrome.tabs.executeScript(tabId, {file: "ytchannel.js", runAt: "document_start"});

It should be js/ytchannel.js, else it is not working.

@tomasko126 tomasko126 self-assigned this Jun 5, 2016
@tomasko126
Copy link
Member

@NuhaKhaled Thanks again for spotting a similar issue. This is a regression of #47, I will fix it in a while.

@tomasko126 tomasko126 added bug This is a bug with the extension pr-high This is a high-priority issue. If you want to help out, try one of these if you can! labels Jun 5, 2016
@tomasko126
Copy link
Member

@NuhaKhaled This issue has been fixed and I will make another release today :)

@YasminRodriguez1
Copy link
Author

Great, Thank you. Good luck with your Catblock.

@tomasko126 tomasko126 added this to the 1.4.1-1.4.x milestone Sep 3, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This is a bug with the extension pr-high This is a high-priority issue. If you want to help out, try one of these if you can!
Projects
None yet
Development

No branches or pull requests

2 participants