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

Android (Kotlin) app template #1867

Merged
merged 17 commits into from
Sep 1, 2023
Merged

Android (Kotlin) app template #1867

merged 17 commits into from
Sep 1, 2023

Conversation

krisbitney
Copy link
Contributor

@krisbitney krisbitney commented Aug 28, 2023

This PR adds an Android app template using the Kotlin app bindings.

Depends on:
#1868
polywrap/wrap-abi-bindgen#53

Ethereum wrapper invocation is not yet working due to incompatibility in Rust wrapscan uri resolver:

uniffi.polywrap_native.FfiException$LoadWrapperException: uri=wrap://ens/wraps.eth:ethereum@1.0.0, err=Failed to resolve wrapper: `Error loading wrapper, uri: wrap://wrapscan.io/polywrap/wrapscan-uri-resolver@1.0: `Failed to resolve wrapper: `Failed to invoke wrapper, uri: `wrap://wrapscan.io/polywrap/http-uri-resolver@1.0`, method: `tryResolveUri`: missing required CPU features: "EnumSet(SSE2 | SSE3 | SSSE3 | SSE41 | SSE42 | POPCNT | AVX | BMI1 | BMI2 | AVX2 | AVX512DQ | AVX512VL | AVX512F | LZCNT)"

@krisbitney krisbitney changed the title kotlin app template Android (Kotlin) app template Aug 29, 2023
@@ -17,7 +17,7 @@ Commands:
wasm [options] <language> <name> Create a Polywrap wasm wrapper. langs:
assemblyscript, rust, golang, interface
app [options] <language> <name> Create a Polywrap application. langs:
typescript
typescript, android
Copy link
Contributor

Choose a reason for hiding this comment

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

Can there be vanilla kotlin or something? Just wondering since the template here is android specific

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We could create another template that is not Android, sure. We have always treated Kotlin as a means of targeting android users, so I think having an Android template makes more sense.

dOrgJelli
dOrgJelli previously approved these changes Sep 1, 2023
@dOrgJelli dOrgJelli merged commit 60ce291 into origin-dev Sep 1, 2023
14 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.

3 participants