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

[#169] [iOS] Add mirrored workflows at the root project to test the iOS app publishing workflows #265

Merged
merged 24 commits into from
Sep 6, 2023

Conversation

doannimble
Copy link
Contributor

@doannimble doannimble commented Aug 7, 2023

What happened 👀

  • Add ios deploy production workflow into root, brick, and sample folder
  • Add APP_STORE_KEY_ID, APP_STORE_ISSUER_ID, APPSTORE_CONNECT_API_KEY_BASE64 to Environments
  • Add lanes: sync_appstore_production_signing , build_and_upload_testflight_production_app, and upload_production_build_to_testflight to fastlane
  • Update upload workflow

Insight 📝

  • To test the changes, please add chore/169-mirrored-workflow-ios as pushed branch, then we can run the flow.

Proof Of Work 📹

Upload to Test Flight

Screenshot 2023-08-14 at 3 16 03 PM

Upload to App Store

Screenshot 2023-08-14 at 3 09 34 PM

@doannimble doannimble self-assigned this Aug 7, 2023
@doannimble doannimble changed the title [#169] [iOS] Add mirrored workflows at the root project to test the iOS app publishing workflows [WIP][#169] [iOS] Add mirrored workflows at the root project to test the iOS app publishing workflows Aug 7, 2023
@doannimble doannimble force-pushed the chore/169-mirrored-workflow-ios branch from 4cb37be to 71add25 Compare August 7, 2023 10:00
@doannimble doannimble changed the base branch from develop to chore/226-update-secrets-management August 7, 2023 10:01
@doannimble doannimble temporarily deployed to production August 11, 2023 07:41 — with GitHub Actions Inactive
@doannimble doannimble force-pushed the chore/169-mirrored-workflow-ios branch from 6b78bb0 to 4e4e688 Compare August 31, 2023 08:53
@luongvo
Copy link
Member

luongvo commented Aug 31, 2023

@doannimble There are still many unresolved reviews from my review round here. Could you please re-check again as you missed all hidden comments?

image

Copy link
Contributor

@manh-t manh-t left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Copy link
Member

@luongvo luongvo left a comment

Choose a reason for hiding this comment

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

@doannimble Some extra issues with formatting & 1 broken result. The rest lgtm.

Copy link
Member

@luongvo luongvo left a comment

Choose a reason for hiding this comment

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

All good now

@luongvo luongvo merged commit 6c98f12 into develop Sep 6, 2023
@luongvo luongvo deleted the chore/169-mirrored-workflow-ios branch September 6, 2023 04:37
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.

[iOS] Add mirrored workflows at the root project to test the iOS app publishing workflows
4 participants