Skip to content

Resolve outdated static folder path creation and default to local share on Linux - #672

Open
GabuTheDev wants to merge 3 commits into
masterfrom
fix/linux-static-default
Open

Resolve outdated static folder path creation and default to local share on Linux#672
GabuTheDev wants to merge 3 commits into
masterfrom
fix/linux-static-default

Conversation

@GabuTheDev

Copy link
Copy Markdown
Member

Reported on discord at https://discord.com/channels/1056534107330445362/1527660136154267800

I also changed the env default to the linux path, because imo it's kind of absurd to have ./static in the .env but find the static folder at ~/.local/share/tosu/static

Fallback for ./static on linux still remains, as I am not willing to do config migration for this little inconvenience.

@GabuTheDev
GabuTheDev requested review from KotRikD and cyperdark July 17, 2026 13:31
@GabuTheDev GabuTheDev self-assigned this Jul 17, 2026
@GabuTheDev GabuTheDev added platform:linux Regarding stuff not present on the windows platform. area:server The nodejs backend of tosu. changelog worthy Has meaning for users; should be mentioned in next changelog. labels Jul 17, 2026
@GabuTheDev
GabuTheDev force-pushed the fix/linux-static-default branch from 7106989 to bd50431 Compare July 18, 2026 22:28
@cyperdark
cyperdark removed their request for review July 19, 2026 01:24
Comment thread packages/common/utils/config.ts
Copilot AI review requested due to automatic review settings July 24, 2026 02:13

This comment was marked as off-topic.

@GabuTheDev
GabuTheDev requested a review from cyperdark July 28, 2026 05:38
@cyperdark
cyperdark requested review from KotRikD and removed request for KotRikD July 28, 2026 06:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:server The nodejs backend of tosu. changelog worthy Has meaning for users; should be mentioned in next changelog. platform:linux Regarding stuff not present on the windows platform.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants