motrix-webextension icon indicating copy to clipboard operation
motrix-webextension copied to clipboard

Not working on Firefox; suggestions

Open thewh1teagle opened this issue 2 years ago • 3 comments

Hello, 2 things The extension does not work on firefox, the extension seems to be broken (see the debug screen shot) image

Another thing - Why do I need to configure manually the secret key for that? This is bad and not comfortable and not following UX best practices. I have Idea for better implement for that motrix app should wait for incoming request for the browser for exchange the secret keys, then the user should accept it from the app. after that the extensions knows the secret and it's much easier and Intuitive way to make the connection between the browser and the app.

Do you look for contributions?

thewh1teagle avatar Nov 21 '21 12:11 thewh1teagle

Are you getting this error on newest version? As for UX sadly for now there is no way for us to talk to the motrix app. I did some research on how we could communicate but haven't gotten around to finishing it and creating pr to motrix repo.

As far as contributions go it is a public repo and contributions are welcome (at least when I make them they get accepted 😄).

JaroslawPokropinski avatar Mar 15 '22 19:03 JaroslawPokropinski

Just as a data point, I am not the poster but have personally been able to get this to work in the latest version when the following criteria are met:

  • The Browser Extension and the Desktop App have the RPC Key set
  • "Prompt Before Download" Is disabled
  • The file meets download size and blacklist criteria

At the moment, unfortunately, the Blacklist and File Size filtering seem to work (e.g. stop from sending a Motrix download request), however they seem to stop the browser download from happening at all. I will open separate issues about this and the Prompt before Download setting, but seemed worthwhile to note that the new updates have enabled a successful download workflow for Firefox

AyoungDukie avatar Oct 16 '22 17:10 AyoungDukie

Also technically this should be closed as a duplicate issue to #59, and a PR could be made for the suggested feature if desired.

AyoungDukie avatar Oct 16 '22 18:10 AyoungDukie