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

feat(ios-deploy): iOS Specific Instructions on Deployments #291

Merged
merged 5 commits into from
Sep 13, 2021

Conversation

michaelachrisco
Copy link
Contributor

@michaelachrisco michaelachrisco commented Sep 9, 2021

Changes

  1. Remove Android Deployment instructions within the iOS deploy page.
  2. Add known good iOS deploy instructions within the deploy page.
  3. Add multiple known good docs within our internal projects for examples.

Purpose

To correct and and more information on iOS deploys. To alleviate some of the pain of deployment.

Approach

For some reason, we had our Android Deployment procedures twice within the repo. I went ahead and removed that as well as added known good instructions with lots of screenshots from one of our internal projects.

Learning

We learned a lot from deploying in both iOS and Android with react native recently. Some of these lessons included making sure everything is documented which should include our S&P repo.

If anyone has any other examples they would like to share, let us know within this PR or parent issue! More examples will make this easier in the future.

Closes #278
Related to #88

@michaelachrisco michaelachrisco self-assigned this Sep 9, 2021
@michaelachrisco michaelachrisco marked this pull request as draft September 9, 2021 22:16
@michaelachrisco michaelachrisco changed the title feat(ios-deploy): ios specific instructions on how to deploy [WIP]: feat(ios-deploy): ios specific instructions on how to deploy Sep 9, 2021
@michaelachrisco michaelachrisco changed the title [WIP]: feat(ios-deploy): ios specific instructions on how to deploy [WIP]: feat(ios-deploy): iOS Specific Instructions on how to Deploy Sep 9, 2021
@michaelachrisco michaelachrisco changed the title [WIP]: feat(ios-deploy): iOS Specific Instructions on how to Deploy [WIP]: feat(ios-deploy): iOS Specific Instructions on Deployments Sep 9, 2021
@michaelachrisco michaelachrisco requested review from a team, JamesVal and aneshamoore and removed request for a team September 9, 2021 23:18
@michaelachrisco michaelachrisco changed the title [WIP]: feat(ios-deploy): iOS Specific Instructions on Deployments feat(ios-deploy): iOS Specific Instructions on Deployments Sep 9, 2021
@michaelachrisco michaelachrisco marked this pull request as ready for review September 9, 2021 23:18
@michaelachrisco
Copy link
Contributor Author

michaelachrisco commented Sep 9, 2021

Added @Basim-Elayan and @Endtry whom both have experience in this field.

- Make sure 'V1' and 'V2' Signature Versions are both checked.
- Click 'finish'
- After it's done, the APK will be available in the APK Destination Folder
1. Open the xcode workspace project.
Copy link
Contributor

Choose a reason for hiding this comment

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

Not sure if we need to mention anything about Signing team, background modes (Capabilities)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for the feedback!

I thought signing happens automatically now within xcode? Or are we talking about a different Signing process?

Do we have a good reference on the background modes? Are you talking about the permissions that we had to set up on the app?

Copy link
Contributor

Choose a reason for hiding this comment

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

Based on the current application required permissions like Bluetooth or remote notifications etc...

Copy link
Contributor

@Basim-Elayan Basim-Elayan Sep 10, 2021

Choose a reason for hiding this comment

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

Here I meant just to mention that with no details as a note. Not sure though if we need it.

Copy link

@Endtry Endtry left a comment

Choose a reason for hiding this comment

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

Good work! Requesting changes to change the order of the steps.

1. Open the xcode workspace project.
2. In upper left-hand of screen, click on `Generic iOS Device`. Select a device you'd like to test on.
3. Build the app via the play button. Be sure the iOS device is set to a physical apple device
<img width="399" alt="Screen Shot 2021-07-30 at 2 23 02 PM" src="https://user-images.githubusercontent.com/25145605/127720045-664ec7b6-95e1-43b5-af55-9938b5c10c5d.png">
Copy link
Contributor

Choose a reason for hiding this comment

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

Very helpful images. Nice job.

Copy link
Contributor Author

@michaelachrisco michaelachrisco Sep 10, 2021

Choose a reason for hiding this comment

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

I believe most of these images were added by @gabymoreno9. We pushed for more docs within the Sundial repo and it did help out later on. I just pulled them all in here and organized a bit.

Copy link
Contributor

@Basim-Elayan Basim-Elayan left a comment

Choose a reason for hiding this comment

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

LGTM, Glad to see these updates in S&P

Copy link

@aneshamoore aneshamoore left a comment

Choose a reason for hiding this comment

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

Everything looks good to me. Nice use of screenshots.

@michaelachrisco michaelachrisco merged commit 55ef563 into main Sep 13, 2021
@michaelachrisco michaelachrisco deleted the mac-278-ios-deploy branch September 13, 2021 20:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

React native iOS deploy instructions include Andoid build/deploy steps
4 participants