joaomgcd / JoinDesktop

A desktop app for Join built in Electron
244 stars 21 forks source link

API link issues #41

Closed mat4444 closed 3 years ago

mat4444 commented 3 years ago

Hi, It's a great feature to embed the API links creation right in the app, but I have noticed a few issues with the 'Create a Join API link for your device' in the desktop app:

It would be great if this could be fixed :) !

mat4444 commented 3 years ago

I don't know if it's directly related to this issue, but when I am pushing a URL to the desktop app via an API link , this does not work: it either open Windows Explorer or opens a Windows dialog box asking which program should open it (when the URL port is defined). However, it works when I send a URL to the Chrome extension or an Android device. It also work when I am using the share option in any webpage from an Android device to the Desktop app.

joaomgcd commented 3 years ago

Fixed the visual issue :) Will be available for the next release.

About the API not working to the Desktop app, can you try sending just with the Text and Title fields and see if that works?

mat4444 commented 3 years ago

The API works when sending only with the text and title fields. However, when I have a URL in the API link, it opens Windows Explorer (or ask which program to use) Sans titre

joaomgcd commented 3 years ago

Can you give me an example of a URL you're using? Thanks!

mat4444 commented 3 years ago

I'm trying to open a URL in the form : xxxxx.duckdns.org:8089. It opens a windows pop-up asking which software should be used to open it. The same API link sent to the chrome extension works perfectly.

Additionally, basic URL like www.google.com, www.youtube.com etc. simply open the Windows explorer.

joaomgcd commented 3 years ago

Does it work if you send with https:// before the URL?

mat4444 commented 3 years ago

Yes, it does work with http:// or https:// ! Thanks ! But it's a bit inconsistent with the behavior of the chrome extension, that does not require it for the first URL example I gave.

joaomgcd commented 3 years ago

The difference is that in Chrome you can't open other kinds of URLs like an app can :) But I guess I can try guessing if the received url is supposed to be an http one...

joaomgcd commented 3 years ago

Ok, changed it :) If there's no protocol in the URL it'll automatically add https://

Hope this helps!

mat4444 commented 3 years ago

Ok cool! Thank you for this update! I will test that!

mat4444 commented 3 years ago

Hi João, when do you think this change will be effective ?

joaomgcd commented 3 years ago

Not sure. I wanted to add some more significant changes before I pushed this out as an update 😅 What version of the app are you using? I can send you the installer directly...

mat4444 commented 3 years ago

No worries! I don't know why, but I thought I would be a change on the server side. I can wait, no problem :) !

joaomgcd commented 3 years ago

ok, cool!