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

Add more chocobo themes #3

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Add more chocobo themes #3

wants to merge 2 commits into from

Conversation

Webberjo
Copy link

Added chocobo themes from Final Fantasy 2, 3, 5, 6, 7, 8, 9, 10 and 14.

Description

Downloaded theme rips off YouTube, imported them into Audacity, clipped them and exported as oggs. When seamless looping wasn't possible, used fade out at end of audio after second loop.

Motivation and Context

Four songs isn't enough for an avid chocobo lover. We need more. More!

How Has This Been Tested?

Logged into WoW TBC Classic, opened Interface, selected Addons -> Chocobo, clicked Reset under Song Files.
Turned music up.
Mounted hawkstrider, dismounted hawkstrider.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have tested my changes in-game.
    • In Retail WoW.
    • In Classic WoW.
    • In Burning Crusade Classic WoW.

@Sharparam Sharparam self-assigned this Nov 15, 2021
@Sharparam
Copy link
Member

My only concern with adding more songs is that it could increase the risk of a DMCA or similar from Square Enix (something similar happened to one of those soundtrack addons in the past if I'm not mistaken).

I approve of the idea of adding more songs, but I'm not sure how it'd work out in the legal sense. Needs a bit of thinking perhaps. Is it more "robust" to keep song files in a separate addon for download? (That could make discovery harder though)

@Webberjo
Copy link
Author

Would it be safer to move the music from the repo to a server and add a button in the addon interface to download them?

@Sharparam
Copy link
Member

AddOns can't perform any file operations in the game, and no communication with external services. So it's not possible to integrate it that way, unfortunately.

There's also the AddOn size which grows a lot with the new songs. This could be another reason to break it into a separate AddOn for download.

I might merge this PR later but hold off on a release for now, while thinking about what might be the best way to go about it. This way the files will be in the repo and can be moved to that other AddOn if that's the path that ends up being the way forward.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants