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

[Audio] Replace Title BGM with custom BGM by Firel #5189

Merged
merged 3 commits into from
Jan 30, 2025
Merged

Conversation

Madmadness65
Copy link
Collaborator

@Madmadness65 Madmadness65 commented Jan 26, 2025

What are the changes the user will see?

Replaces the PMD Top Menu music with a custom track made by Firel.

Why am I making these changes?

The PMD title music was a temporary replacement until we got another unique title theme again.

What are the changes from a developer perspective?

Replaced audio file, added a loop point entry for the title track, as well as added some documentation for the getBgmLoopPoint methods in the battle scene and arena files.

Screenshots/Videos

title.mp4

How to test the changes?

Go to the main menu.

Checklist

  • I'm using beta as my base branch
  • There is no overlap with another PR?
  • The PR is self-contained and cannot be split into smaller PRs?
  • Have I provided a clear explanation of the changes?
  • Have I tested the changes manually?
  • Are all unit tests still passing? (npm run test)
    - [ ] Have I created new automated tests (npm run create-test) or updated existing tests related to the PR's changes?
  • Have I provided screenshots/videos of the changes (if applicable)?
    - [ ] Have I made sure that any UI change works for both UI themes (default and legacy)?

Are there any localization additions or changes? If so:

Also added descriptions for the battle scene and arena's getBgmLoopPoint methods.
damocleas
damocleas previously approved these changes Jan 26, 2025
Adri1
Adri1 previously approved these changes Jan 26, 2025
DayKev
DayKev previously approved these changes Jan 26, 2025
@DayKev DayKev added the Awaiting Locales This PR makes changes to existing locales and requires further review label Jan 26, 2025
@Madmadness65 Madmadness65 dismissed stale reviews from DayKev, Adri1, and damocleas via 8a2ae0a January 26, 2025 23:02
@damocleas damocleas merged commit ec09186 into beta Jan 30, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Audio Relates to the SFX or BGM Awaiting Locales This PR makes changes to existing locales and requires further review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants