Skip to content

Replace --persist flag with REPLACE_PLAYLIST .env, Remove stale persist toggles#191

Merged
LumePart merged 9 commits into
LumePart:devfrom
dammitjeff:remove-stale-persist-toggles
Jul 16, 2026
Merged

Replace --persist flag with REPLACE_PLAYLIST .env, Remove stale persist toggles#191
LumePart merged 9 commits into
LumePart:devfrom
dammitjeff:remove-stale-persist-toggles

Conversation

@dammitjeff

@dammitjeff dammitjeff commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

PERSIST was incredibly confusing for users and devs to understand whats going on, and made it difficult to debug features since true meant "don't delete" false meant "delete and replace." The UI already had to double-invert just for it to make sense for end-users. Not good lol.

Swapped out the --persist flag with a new env, and wired it up to the settings.jsx page.

REPLACE_PLAYLIST=true - Delete old playlist, Create new one (this is now default)
REPLACE_PLAYLIST=false - Keep old playlist, Create new one alongside old playlist

Also added a one time migration in server.go to help convert existing "persist" values for anyone that was already using it, on startup. Will delete at a later date once people are migrated over.

Also removed stale --persist toggles in the frontend as they're now redundant with the toggles in settings.

Comment thread src/config/config.go
@dammitjeff
dammitjeff force-pushed the remove-stale-persist-toggles branch from 959690b to 424f224 Compare July 13, 2026 06:17
@dammitjeff

dammitjeff commented Jul 13, 2026

Copy link
Copy Markdown
Contributor Author

Hey! Made a few updates now that the big refactor is completed :)

I removed the env and switched back to a --replace_playlistflag. The default is now to overwrite the playlist unless the run has --replace-playlist=false to better reflect users behavior.

The frontend his now also wired up in place now, and keeping it as a flag means now it can be toggled on a per-playlist basis!
Screenshot 2026-07-13 at 12 19 49 AM
A description of what the toggle does appears on hover of the info icon. :)

Also removed the old "Update playlist in place" setting in the settings page as it's now redundant.

I'm ready for merge now to upstream but, definitely let me know if you have questions! I tested this pretty thoroughly and now it works REALLY nice. From the user's end, it finally feels like an actual self hosted auto updated playlist.

Screenshot 2026-07-13 at 1 00 08 AM

@LumePart

Copy link
Copy Markdown
Owner

Thanks for thew PR :)

@LumePart
LumePart merged commit 1e7a7ca into LumePart:dev Jul 16, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants