Skip to content

Conversation

fdnt7
Copy link
Member

@fdnt7 fdnt7 commented Jun 10, 2025

Rebuild lyra_proc crate if .env is changed, left out of #64. This makes the /play sources to be reevaluated every time .env changes, instead of incorrectly on every cold build.

Rebuild `lyra_proc` crate if `.env` is changed, left out of #64. This
makes the `/play` sources to be reevaluated every time `.env` changes,
instead of incorrectly on every cold build.
@fdnt7 fdnt7 added this to the v0.9.1 milestone Jun 10, 2025
@fdnt7 fdnt7 self-assigned this Jun 10, 2025
@fdnt7 fdnt7 merged commit a0794e0 into main Jun 10, 2025
3 checks passed
@fdnt7 fdnt7 deleted the bugfix/proc-rerun-if-changed-dotenv branch June 10, 2025 07:05
fdnt7 added a commit that referenced this pull request Jun 18, 2025
Sync `/play`'s 'source' option choices with enabled play sources enabled
in `.env` at runtime instead of build time. This allows prebuilt images
to declare the new source option choices when `.env` is changed
accordingly.

refs: #64, #77
fdnt7 added a commit that referenced this pull request Jun 19, 2025
* fix: sync /play src opt choices w/ .env at runtime

Sync `/play`'s 'source' option choices with enabled play sources enabled
in `.env` at runtime instead of build time. This allows prebuilt images
to declare the new source option choices when `.env` is changed
accordingly.

refs: #64, #77

* fix: include display_name in runtime .env syncing

Also refactor string literals out to their own named consts which should
be more extensible.
fdnt7 added a commit that referenced this pull request Jun 19, 2025
Pass missing environment variables the bot image needs in the example
docker compose file: `PLUGINS_LAVASRC_SOURCES_DEEZER`,
`PLUGINS_LAVASRC_SOURCES_SPOTIFY` and `PLUGINS_YOUTUBE_OAUTH_ENABLED`.

refs: #64, #66, #77, #90
fdnt7 added a commit that referenced this pull request Jun 19, 2025
Pass missing environment variables the bot image needs in the example
docker compose file: `PLUGINS_LAVASRC_SOURCES_DEEZER`,
`PLUGINS_LAVASRC_SOURCES_SPOTIFY` and `PLUGINS_YOUTUBE_OAUTH_ENABLED`.

refs: #64, #66, #77, #90
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.

1 participant