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

docs: add Capawesome Cloud integration #2503

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

robingenz
Copy link

@robingenz robingenz commented May 2, 2024

This PR adds instructions on how to integrate Capawesome Cloud with Codemagic.

Related to codemagic-ci-cd/codemagic-sample-projects#192.

@robingenz
Copy link
Author

Is there anything I need to update?

npx capawesome apps:bundles:create --appId <app-id> --path <path-to-bundle>
{{< /highlight >}}

Make sure to replace `<app-id>` with the ID of your app on Capawesome Cloud and `<path-to-bundle>` with the path to the bundle you want to upload (e.g. `www`).
Copy link
Collaborator

@icarusdust icarusdust Nov 7, 2024

Choose a reason for hiding this comment

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

thanks for the PR

perhaps better show the actual bundle path where e.g. ios binary is located: build/ios/ipa/App.ipa

Copy link
Collaborator

@icarusdust icarusdust Nov 7, 2024

Choose a reason for hiding this comment

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

or remove the e.g. 'www' part because as i understand you need to upload ios and android binaries here

Copy link
Author

Choose a reason for hiding this comment

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

Thanks for getting back! No, you do not need to upload the Android and iOS binaries. It's not possible to update the native version of an app via live updates. You can only update the web assets of an app. For this reason, --path actually refers to the path of the web build (e.g. www or dist).

Copy link
Collaborator

Choose a reason for hiding this comment

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

thanks for clarifying it.

Copy link
Collaborator

@icarusdust icarusdust Nov 8, 2024

Choose a reason for hiding this comment

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

I added a note in the sample project, after making those changes, it looks good to be merged.

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