Skip to content

PurchaseTester: setting changelog when submitting to TestFlight#2232

Merged
NachoSoto merged 1 commit into
mainfrom
purchase-tester-testflight-changelog
Jan 18, 2023
Merged

PurchaseTester: setting changelog when submitting to TestFlight#2232
NachoSoto merged 1 commit into
mainfrom
purchase-tester-testflight-changelog

Conversation

@NachoSoto

@NachoSoto NachoSoto commented Jan 18, 2023

Copy link
Copy Markdown
Contributor

This will provide some context when a new build is submitted.
Also changed the lane to use upload_to_testflight instead of upload_to_appstore.

IMG_C78B738B91AB-1

@NachoSoto NachoSoto added the ci label Jan 18, 2023
@NachoSoto NachoSoto requested a review from a team January 18, 2023 20:10
Comment thread fastlane/Fastfile Outdated

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This was slightly wrong, now it works when called from any folder.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

yeah, my bad, I didn't really consider paths other than calling from root. Solid improvement

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It wasn't used from any of the testers before :)

@NachoSoto NachoSoto force-pushed the purchase-tester-testflight-changelog branch 2 times, most recently from 93c4d3e to 7d35471 Compare January 18, 2023 20:22
This will provide some context when a new build is submitted.
Also changed the lane to use `upload_to_testflight` instead of `upload_to_appstore`.
Comment thread .circleci/config.yml Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

all of these are coming back before merging, right?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Removed commit 👍🏻 was making sure it was still working.

Comment thread fastlane/Fastfile Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

yeah, my bad, I didn't really consider paths other than calling from root. Solid improvement

@NachoSoto NachoSoto force-pushed the purchase-tester-testflight-changelog branch from 7d35471 to 2474812 Compare January 18, 2023 20:40
@NachoSoto NachoSoto enabled auto-merge (squash) January 18, 2023 20:41
@NachoSoto NachoSoto merged commit 995eba2 into main Jan 18, 2023
@NachoSoto NachoSoto deleted the purchase-tester-testflight-changelog branch January 18, 2023 20:49
NachoSoto added a commit that referenced this pull request Jan 18, 2023
NachoSoto pushed a commit that referenced this pull request Jan 19, 2023
**This is an automatic release.**

### Bugfixes
* `Purchases`: avoid potential crash when initializing in the background
(#2231) via NachoSoto (@NachoSoto)
### Other Changes
* `PurchaseTester`: ignore errors when restoring purchases (#2228) via
NachoSoto (@NachoSoto)
* `PurchaseTester`: fixed `isPurchasing` state when purchasing fails
(#2229) via NachoSoto (@NachoSoto)
* `PurchaseTester`: setting `changelog` when submitting to `TestFlight`
(#2232) via NachoSoto (@NachoSoto)
* Revert "`SPM`: added `APPLICATION_EXTENSION_API_ONLY` flag to
`RevenueCat` and `ReceiptParser` (#2217)" (#2225) via NachoSoto
(@NachoSoto)
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