Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix issues with main menu music script #748

Merged
merged 2 commits into from May 19, 2024

Conversation

treacherousfiend
Copy link
Contributor

Should've noticed these issues before, but I only double checked the PR after it was merged.

-Changed comment in comfig.cfg to actually say what it does
-Fixed dynamic_background=dustbowl to play music (Although after testing it, it is a bit weird hearing music and the soundscapes of background01. Up to taste, maybe worth having a "no music" option?)
-Fixed dynamic_background=preload setting dynamic_background_level to echo dynamic_background=itemtest
-Fixed typo causing Taps to not play during Soldier holiday
-Added .nut files to the cleanItems function in dev/common.sh to reduce filesize. I have 0 experience with this so I just added these to the pre-existing cleaning functions. It DOES NOT clean whitespace! also there's some places in the file that need a bit of whitespace, so a naive script won't work.

-Fixed dynamic_background=dustbowl to play music (Although after testing it, it is a bit weird hearing music and the soundscapes of background01. Up to taste, maybe worth having a "no music" option?)
-Fixed dynamic_background=preload setting `dynamic_background_level` to `echo dynamic_background=itemtest`
-Fixed typo causing Taps to not play during Soldier holiday.
doesn't clean whitespace. can a linux user come fix this please
@mastercoms mastercoms merged commit 4a016b4 into mastercomfig:develop May 19, 2024
2 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.

None yet

2 participants