Skip to content

Comments

Android basics#69

Merged
gewenyu99 merged 13 commits intomainfrom
android
Feb 10, 2026
Merged

Android basics#69
gewenyu99 merged 13 commits intomainfrom
android

Conversation

@gewenyu99
Copy link
Collaborator

@gewenyu99 gewenyu99 commented Feb 5, 2026

Will need extensive testing but here it is in its basic form

This PR is just the example, not adding it to the skills yet until some more testing

Screen.Recording.2026-02-05.at.1.08.29.PM.mov

@github-actions
Copy link

github-actions bot commented Feb 5, 2026

🧙 Wizard CI

Run the Wizard CI and test your changes against wizard-workbench example apps by replying with a GitHub comment using one of the following commands:

Test all apps:

  • /wizard-ci all

Test all apps in a directory:

  • /wizard-ci django
  • /wizard-ci flask
  • /wizard-ci laravel
  • /wizard-ci next-js
  • /wizard-ci react-router

Test an individual app:

  • /wizard-ci django/django3-saas
  • /wizard-ci flask/flask3-social-media
  • /wizard-ci laravel/laravel12-saas
Show more apps
  • /wizard-ci next-js/15-app-router-saas
  • /wizard-ci next-js/15-app-router-todo
  • /wizard-ci next-js/15-pages-router-saas
  • /wizard-ci next-js/15-pages-router-todo
  • /wizard-ci react-router/react-router-v7-project
  • /wizard-ci react-router/rrv7-starter
  • /wizard-ci react-router/saas-template
  • /wizard-ci react-router/shopper

Results will be posted here when complete.

@gewenyu99 gewenyu99 requested a review from a team February 5, 2026 19:30
@edwinyjlim
Copy link
Member

you're a god among men

@gewenyu99
Copy link
Collaborator Author

For the record, we can't not export the main activity. It's literally impossible to launch then.

https://developer.android.com/privacy-and-security/risks/android-exported

@gewenyu99
Copy link
Collaborator Author

Confirmed this stuff now works. The skill file is a little larger than I'd like, but doesn't have anything that shouldn't be here:
android.zip

Copy link
Member

@edwinyjlim edwinyjlim left a comment

Choose a reason for hiding this comment

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

yahoooooo

we might need some commandments since this is our first step into Java-land

Copy link
Member

Choose a reason for hiding this comment

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

gotta add to root .gitignore

Comment on lines 72 to 73
implementation(libs.androidx.navigation.compose)
testImplementation(libs.junit)
Copy link
Member

Choose a reason for hiding this comment

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

missing posthog SDK?

Suggested change
implementation(libs.androidx.navigation.compose)
testImplementation(libs.junit)
implementation(libs.androidx.navigation.compose)
implementation(libs.posthog.android)
testImplementation(libs.junit)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Where'd it go? LOL how

Comment on lines +13 to +14
[libraries]
androidx-core-ktx = { group = "androidx.core", name = "core-ktx", version.ref = "coreKtx" }
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
[libraries]
androidx-core-ktx = { group = "androidx.core", name = "core-ktx", version.ref = "coreKtx" }
[libraries]
posthog-android = { group = "com.posthog", name = "posthog-android", version.ref = "posthog" }
androidx-core-ktx = { group = "androidx.core", name = "core-ktx", version.ref = "coreKtx" }

activityCompose = "1.8.0"
kotlin = "2.0.21"
composeBom = "2024.09.00"
navigationCompose = "2.7.5"
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
navigationCompose = "2.7.5"
navigationCompose = "2.7.5"
posthog = "3.31.0"

@gewenyu99
Copy link
Collaborator Author

addressed comments + added commandments.

@gewenyu99 gewenyu99 merged commit 7a1f43f into main Feb 10, 2026
8 of 9 checks passed
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.

2 participants