jellyfin / jellyfin.org

The Jellyfin website, blog, and user documentation
http://jellyfin.org
Other
82 stars 292 forks source link

refactor(web-config): remove mention to stable config as config.json source #1113

Open ferferga opened 3 weeks ago

ferferga commented 3 weeks ago

This is out of date already and with the faster cadence of releases we plan to have, this can easily become a burden to maintain.

@thornbill Does web break if the json file contains keys that it's not aware of (that's the only incompatibility I can see, some new setting introduced in an update)? Or it's able to just pick the ones that it expects correctly?

Marking as a draft until we have OK from Bill in that regard

jellyfin-bot commented 3 weeks ago

Cloudflare Pages deployment

Latest commit f673098c03724791faf8e73159dfe661a590d554
Status ✅ Deployed!
Preview URL https://beee1a39.jellyfin-org.pages.dev
Type 🔀 Preview
thornbill commented 1 week ago

Changes to (web) plugins will likely be problematic (cause failed imports).

ferferga commented 1 week ago

@thornbill Any chance the wrong imports can gracefully fail so it doesn't matter or that's a difficult thing to tackle right now?