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 project settings for AVAudioSessionCategory on iOS #81196

Merged
merged 1 commit into from
Oct 17, 2023

Conversation

georgwacker
Copy link
Contributor

@georgwacker georgwacker commented Aug 31, 2023

Adds project settings for AVAudioSessionCategory on iOS, related to #50638 (does not fix the error log).

session_category

For the setting path I tried to stay consistent with the setting structure in display/window/ios.

@georgwacker georgwacker requested review from a team as code owners August 31, 2023 13:41
platform/ios/app_delegate.mm Outdated Show resolved Hide resolved
platform/ios/app_delegate.mm Outdated Show resolved Hide resolved
@georgwacker georgwacker force-pushed the add-audiosession-options branch 3 times, most recently from 344a5c6 to 91b92b5 Compare August 31, 2023 14:13
@AThousandShips
Copy link
Member

Also documentation has to be sorted alphabetically

doc/classes/ProjectSettings.xml Outdated Show resolved Hide resolved
doc/classes/ProjectSettings.xml Outdated Show resolved Hide resolved
doc/classes/ProjectSettings.xml Outdated Show resolved Hide resolved
platform/ios/app_delegate.mm Outdated Show resolved Hide resolved
doc/classes/ProjectSettings.xml Outdated Show resolved Hide resolved
@georgwacker
Copy link
Contributor Author

I changed the setting to int and created an enum for the comparison, let me know if that is proper or if the enum should reside in the Godot context? I did not find an example in the macos/ios code that uses an enum, so I'm not sure how I would access it.

core/config/project_settings.cpp Outdated Show resolved Hide resolved
Copy link
Member

@AThousandShips AThousandShips left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Style looks good

@akien-mga akien-mga modified the milestones: 4.x, 4.2 Oct 17, 2023
@akien-mga akien-mga merged commit e6d488f into godotengine:master Oct 17, 2023
15 checks passed
@akien-mga
Copy link
Member

Thanks!

@georgwacker georgwacker deleted the add-audiosession-options branch October 17, 2023 15:18
@akien-mga akien-mga changed the title Add project settings for AVAudioSessionCategory on iOS Add project settings for AVAudioSessionCategory on iOS Jan 30, 2024
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.

4 participants