-
Notifications
You must be signed in to change notification settings - Fork 240
Enable Web support for demo-compose-app #780
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
Enable Web support for demo-compose-app #780
Conversation
koin-bom = "4.1.1" | ||
koog = "0.4.1" | ||
kotlin = "2.2.10" | ||
kotlin = "2.2.20" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Kotlin 2.2.20 introduce common webMain source set for js/wasmJs targets.
KT-79489 Generate Stdlib API reference for webMain source set
4c2a6eb
to
73bf0c2
Compare
73bf0c2
to
1606875
Compare
@kpavlov I have updated the PR:
Jvm, Android, iOS and Web (js/wasmJs) targets work as expected for me locally. |
Thank you @siarhei-luskanau |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, but can't verify how it works now
implementation(libs.ktor.client.darwin) | ||
} | ||
|
||
webMain.dependencies { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need any KTor client dependency specifically for web?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good question. Looks like default CIO Ktor engine is used for web. Actually it works without any Ktor dependency declarations for me .
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Checked:
- iOS App
- Android App
- Desktop App
- Web App
assembling and work. Thank you so much for the contribution!
This commit introduces WebAssembly (Wasm) and JS browser support to the demo-compose-app.
1606875
to
fb1baab
Compare
This commit introduces WebAssembly (Wasm) and JS browser support to the demo-compose-app. <!-- Thank you for opening a pull request! Please add a brief description of the proposed change here. Also, please tick the appropriate points in the checklist below. --> ## Motivation and Context <!-- Why is this change needed? What problem does it solve? --> ## Breaking Changes <!-- Will users need to update their code or configurations? --> --- #### Type of the changes - [ ] New feature (non-breaking change which adds functionality) - [ ] Bug fix (non-breaking change which fixes an issue) - [ ] Breaking change (fix or feature that would cause existing functionality to change) - [x] Documentation update - [ ] Tests improvement - [x] Refactoring #### Checklist - [x] The pull request has a description of the proposed change - [x] I read the [Contributing Guidelines](https://github.com/JetBrains/koog/blob/main/CONTRIBUTING.md) before opening the pull request - [x] The pull request uses **`develop`** as the base branch - [x] Tests for the changes have been added - [x] All new and existing tests passed ##### Additional steps for pull requests adding a new feature - [ ] An issue describing the proposed change exists - [ ] The pull request includes a link to the issue - [ ] The change was discussed and approved in the issue - [ ] Docs have been added / updated
This commit introduces WebAssembly (Wasm) and JS browser support to the demo-compose-app.
Motivation and Context
Breaking Changes
Type of the changes
Checklist
develop
as the base branchAdditional steps for pull requests adding a new feature