Custom protocol fix (tidal://...)

[TheRockYT]

PR for issue #138 (tidal:// protocol)

The custom protocol ('tidal://...') wasn't working correctly:

• If there was no instance running or single-instance disabled, a window opened on the main page ('https://listen.tidal.com/')
• If there was already a window open and single-instance enabled, nothing happens.

What I've changed

• Now, the app checks for the presence of the custom protocol request before the usual URL ('https://listen.tidal.com/') is loaded.
• In single-instance mode, the main-app opens the page requested by the second-instance.

What could be improved

• In singe-instance mode, the whole page gets reloaded, so the playback stops. It should be possible to use TIDAL's router to prevent it.
• Multi-instance mode is still very buggy on my tests: Random logouts, page not loading, ... (idk. why it is useful to have TIDAL multiple times open)

How to test

To test the custom protocol, the app needs to be installed locally (deb, rpm, ...) and run at least one time.

For example, to build a deb just run npm run build-deb and install with sudo apt reinstall ./dist/tidal-hifi_*.deb

Important: The electron test command does not let the registration of custom protocols happen.

Then, go to an artist, playlist or album page on the TIDAL web-page (not web-player) and click "Open in Desktop App".

Example TIDAL web-page links:

• Lady Gaga on TIDAL - Artist
• Taylor Swift on TIDAL - Artist
• TIDAL Top 100 Germany - Playlist
• ...

I did not test this on other devices then on an Ubuntu-Desktop and a Fedora-Laptop, so please report issues with them.

[sonarcloud[bot]]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

[Mastermindzh]

Hey @TheRockYT,

Thanks for the PR, initial code looks good. I'm able to test it during the weekend (probably) and based on that we can continue with either a merge or maybe some more bugfixing.

Multiple TIDAL windows and not "correctly" handling the protocol doesn't really matter to me, kinda weird to have 2 open anyway as you already point out. I mean... I'll have a little look at it, but probably fine.