Arlodotexe / strix-music

Combine any music sources into a single library. It's your music. Play it your way.
http://www.strixmusic.com
136 stars 4 forks source link

[new] Added ipfs as a media source #278

Closed amaid closed 10 months ago

amaid commented 1 year ago

Overview

Under media source, currently we only support local file and OneDrive, this improvement will add another media source i.e. IPFS media source.

Closes #242

Following the approach of OneDrive media source, the FileBrowser and FileSelector was reused to show the folders from ipfs. Currently root folder selection using ipns address disabled.

Checklist

This PR meets the following requirements:

Additional info

Not provided

Arlodotexe commented 12 months ago

Found a few UI issues related to folder browser. Namely that you can't select the folder the user provided in the previous step.

Maybe we just get rid of the folder browser here and just use the path provided by the user, then move the unDraw graphic to the CID/IPNS entry instead.

image

Arlodotexe commented 12 months ago

I've also noted that DNSLink addresses don't work for IPNS? The IpnsFolder in OwlCore.Storage should be able to handle both of these:

Arlodotexe commented 12 months ago

Upon adding this new music source, nothing happens:

https://github.com/Arlodotexe/strix-music/assets/9384894/0968e7eb-8a9f-40b0-a004-a6bcf048631e

amaid commented 11 months ago

Found a few UI issues related to folder browser. Namely that you can't select the folder the user provided in the previous step.

Maybe we just get rid of the folder browser here and just use the path provided by the user, then move the unDraw graphic to the CID/IPNS entry instead.

image

We decided to remove this screen and directly pick the Cid from the root directory screen.