Skip to content

Add AudioFlac Variant to Support FLAC Uploads#65

Merged
Sleitnick merged 1 commit intoSleitnick:mainfrom
Zidiam:main
Jan 13, 2025
Merged

Add AudioFlac Variant to Support FLAC Uploads#65
Sleitnick merged 1 commit intoSleitnick:mainfrom
Zidiam:main

Conversation

@Zidiam
Copy link
Copy Markdown
Contributor

@Zidiam Zidiam commented Jan 13, 2025

This pull request introduces a new AudioFlac variant to the AssetType enum, enabling .flac uploads through the Roblox Open Cloud. The changes include:

  • Adding the AudioFlac variant to AssetType.
  • Mapping .flac to audio/flac in content_type.
  • Updating asset_type and try_from_extension to treat .flac as audio.

This allows rbxcloud to recognize FLAC files and forward them to Roblox’s asset creation APIs without throwing an “unknown extension” error.

@Sleitnick
Copy link
Copy Markdown
Owner

Thanks! Have you tested this out and confirmed it works? Asking because the docs for the Asset API do not include .flac files as supported. The actual upload prompt on the website does show it though.

@Zidiam
Copy link
Copy Markdown
Contributor Author

Zidiam commented Jan 13, 2025

@Sleitnick Yes, but I tested using asphalt, and saw the asset get generated with everything being successful.

@Zidiam
Copy link
Copy Markdown
Contributor Author

Zidiam commented Jan 13, 2025

@Sleitnick Here is my asphalt MR:
jackTabsCode/asphalt#91

@Sleitnick Sleitnick merged commit 252ca2a into Sleitnick:main Jan 13, 2025
@jackTabsCode
Copy link
Copy Markdown
Contributor

jackTabsCode commented Jan 13, 2025

I noticed that they started supporting more formats for manual upload but didn't document them as supported on the Open Cloud APIs. Strange.

@Zidiam Have you tried to play back this audio in a game?

@Zidiam
Copy link
Copy Markdown
Contributor Author

Zidiam commented Jan 16, 2025

@jackTabsCode Ya attached is a screenshot of .flac and .wav audio.
image
image

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.

3 participants