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 CATEGORY_UNDEFINED option in Android export #98971

Merged
merged 1 commit into from
Nov 12, 2024

Conversation

syntaxerror247
Copy link
Contributor

@syntaxerror247 syntaxerror247 commented Nov 8, 2024

The app category is optional in Android, but currently, there is no option in the Godot editor to leave it undefined. In this PR, I have introduced a new category, UNDEFINED. When this category is selected, the appCategory will not be defined in the Android manifest file.

I have implemented logic to remove this attribute completely during the Gradle build export. However, in the normal export, I am setting its value to -1 instead.

@syntaxerror247 syntaxerror247 requested a review from a team as a code owner November 8, 2024 16:42
@AThousandShips AThousandShips added this to the 4.x milestone Nov 8, 2024
@syntaxerror247 syntaxerror247 changed the title Add new android app_category other Add 'OTHER' Category Option to Leave App Category Undefined in Android Manifest Nov 8, 2024
@syntaxerror247 syntaxerror247 changed the title Add 'OTHER' Category Option to Leave App Category Undefined in Android Manifest Add 'OTHER' category option in Android export Nov 8, 2024
@syntaxerror247 syntaxerror247 marked this pull request as draft November 8, 2024 16:53
platform/android/export/gradle_export_util.h Outdated Show resolved Hide resolved
platform/android/export/export_plugin.cpp Outdated Show resolved Hide resolved
platform/android/export/export_plugin.cpp Outdated Show resolved Hide resolved
@syntaxerror247 syntaxerror247 changed the title Add 'OTHER' category option in Android export Add 'UNDEFINED' category option in Android export Nov 8, 2024
@syntaxerror247 syntaxerror247 changed the title Add 'UNDEFINED' category option in Android export Add CATEGORY_UNDEFINED option in Android export Nov 8, 2024
@syntaxerror247 syntaxerror247 marked this pull request as ready for review November 8, 2024 17:57
Copy link
Contributor

@m4gr3d m4gr3d left a comment

Choose a reason for hiding this comment

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

Looks good!

@m4gr3d m4gr3d modified the milestones: 4.x, 4.4 Nov 8, 2024
@Repiteo Repiteo merged commit b6bbd40 into godotengine:master Nov 12, 2024
20 checks passed
@Repiteo
Copy link
Contributor

Repiteo commented Nov 12, 2024

Thanks!

@syntaxerror247 syntaxerror247 deleted the new_appCategory branch November 12, 2024 15:50
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