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

"Download file" stopped working #61

Open
ThibaultVlacich opened this issue Aug 8, 2024 · 8 comments
Open

"Download file" stopped working #61

ThibaultVlacich opened this issue Aug 8, 2024 · 8 comments

Comments

@ThibaultVlacich
Copy link

ThibaultVlacich commented Aug 8, 2024

Extension stopped working, don't really know why. When I click the Screenshot button, the file does not get saved.

I enabled the logging, I see this in the console when I click the button:

Youtube Screenshot Addon: Successfully started download file

But no file is getting downloaded. Firefox 129.0 released two days ago. I feel like it was working just before that, so maybe related?

EDIT: copy to clipboard works fine.

@ThibaultVlacich ThibaultVlacich changed the title Stopped working "Download file" stopped working Aug 8, 2024
@UncookedMoa
Copy link

UncookedMoa commented Aug 9, 2024

I am also experiencing this issue, and have found that this seems to only be an issue for specific videos. Some videos don't seem to be able to be screenshotted at all, while others can. Really bizarre.

@gurumukhi
Copy link
Owner

Thank you very much @ThibaultVlacich and @UncookedMoa for reporting this issue. I think there is an issue in the latest version 4.3.0 which was released this month. For now I have disabled this version and enabled previous version (4.2.0).

@gurumukhi
Copy link
Owner

Even after reverting back to previous version there seems to be some issues (as per latest reviews). Is this because of YouTube UI changes or some issue in our addon itself?

@joggee-fr

@joggee-fr
Copy link
Contributor

I don't know how versions are managed in AMO. @gurumukhi, did you know if an existing installation is automatically downgraded in such a case?
Maybe you should revert the PR here and create a new version to be sure. I can't inspect the issue right now.

@gurumukhi
Copy link
Owner

Version management on AMO is simple - You can keep uploading new versions and whenever a new version is causing an issue, we can simply disable that version to enable previous one. In this screenshot, you can see I simply disabled 4.3.0 version and now 4.2.0 version is enabled.
AMO-versions

You can download current and old version files from https://addons.mozilla.org/en-US/firefox/addon/youtube-screenshot-button/versions/.
So, right now the version enabled on the AMO is 4.2.0 and you can check that code at https://github.com/gurumukhi/youtube-screenshot/tree/f09a74e0cae3124d667c170367caeb0bf2b49a04.

@ThibaultVlacich
Copy link
Author

It does not auto downgrade indeed. I had to uninstall the extension and re-install it to get the not bugged version.

Maybe you can re-publish the working version as a 4.4.0 ?

@joggee-fr
Copy link
Contributor

@ThibaultVlacich @UncookedMoa
What it is the behavior you observe when trying to download the screenshot as a file? No file is really saved in your download directory?

@joggee-fr
Copy link
Contributor

Gentle ping @ThibaultVlacich @UncookedMoa.

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

4 participants