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

Update Swift example to use SwiftUI #2416

Merged
merged 2 commits into from
Feb 28, 2024

Conversation

keith
Copy link
Member

@keith keith commented Feb 28, 2024

Fixes #2414

Copy link
Contributor

@mattrobmattrob mattrobmattrob left a comment

Choose a reason for hiding this comment

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

👋

@keith keith merged commit f8fa4d9 into master Feb 28, 2024
9 of 10 checks passed
@keith keith deleted the ks/update-swift-example-to-use-swiftui branch February 28, 2024 18:06
mattrobmattrob pushed a commit that referenced this pull request Mar 23, 2024
## Related Issue/PR

#2416 cc @keith 

## Modification

Update example's Info.plist file.

> Open the newly created file and paste in it the contents found in this
[example Info.plist
file](https://github.com/bazelbuild/rules_apple/blob/master/examples/ios/HelloWorldSwift/Info.plist).

The
[tutorial](https://github.com/bazelbuild/rules_apple/blob/master/doc/tutorials/ios-app.md)
guide us to use the Info.plist file here.

But it lacks some information and will make the ipa binary to launch as
compatible mode of legacy iOS app.

## Description

Before the PR:
![Old
Behavior](https://github.com/bazelbuild/rules_apple/assets/43724855/d811d90c-1bee-4134-ba93-082ffb66bd9a)

After the PR:
![New
Behavior](https://github.com/bazelbuild/rules_apple/assets/43724855/e9d20d09-26c1-4a56-9f35-1efec7e3fc42)

## Test

You can verify the behavior by running the
[tutorial](https://github.com/bazelbuild/rules_apple/blob/master/doc/tutorials/ios-app.md).

Or

Once we delete the extra `launch_storyboard =
"//examples/resources:Launch.storyboard"` in `HelloWorldSwift`. We can
also reproduce it by running the following commands

```shell
cd ${PROJECT_ROOT}/examples/ios/HelloWorldSwift
# If you are running a stable bazel, you can `rm ${PROJECT_ROOT}/.bazelversion` to workaround the error
bazel run //examples/ios/HelloWorldSwift:HelloWorldSwift 
```
acecilia pushed a commit to revolut-mobile/rules_apple that referenced this pull request Apr 12, 2024
acecilia pushed a commit to revolut-mobile/rules_apple that referenced this pull request Apr 12, 2024
## Related Issue/PR

bazelbuild#2416 cc @keith 

## Modification

Update example's Info.plist file.

> Open the newly created file and paste in it the contents found in this
[example Info.plist
file](https://github.com/bazelbuild/rules_apple/blob/master/examples/ios/HelloWorldSwift/Info.plist).

The
[tutorial](https://github.com/bazelbuild/rules_apple/blob/master/doc/tutorials/ios-app.md)
guide us to use the Info.plist file here.

But it lacks some information and will make the ipa binary to launch as
compatible mode of legacy iOS app.

## Description

Before the PR:
![Old
Behavior](https://github.com/bazelbuild/rules_apple/assets/43724855/d811d90c-1bee-4134-ba93-082ffb66bd9a)

After the PR:
![New
Behavior](https://github.com/bazelbuild/rules_apple/assets/43724855/e9d20d09-26c1-4a56-9f35-1efec7e3fc42)

## Test

You can verify the behavior by running the
[tutorial](https://github.com/bazelbuild/rules_apple/blob/master/doc/tutorials/ios-app.md).

Or

Once we delete the extra `launch_storyboard =
"//examples/resources:Launch.storyboard"` in `HelloWorldSwift`. We can
also reproduce it by running the following commands

```shell
cd ${PROJECT_ROOT}/examples/ios/HelloWorldSwift
# If you are running a stable bazel, you can `rm ${PROJECT_ROOT}/.bazelversion` to workaround the error
bazel run //examples/ios/HelloWorldSwift:HelloWorldSwift 
```
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.

iOS app tutorial steps produce an app that crashes on launch
3 participants