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

Removes api key from integration test #1098

Merged
merged 1 commit into from
Jun 14, 2024
Merged

Removes api key from integration test #1098

merged 1 commit into from
Jun 14, 2024

Conversation

JayShortway
Copy link
Member

As the title says. Saw this while working on something else.

Reading .circleci/config.yml it seems that the replace-api-key command should pick this up and replace it properly, but let me know if I'm overlooking something!

I'm also happy to close this if this is not a problem, or if the fix is not as easy as this PR.

@JayShortway JayShortway requested a review from a team June 13, 2024 15:11
@JayShortway JayShortway self-assigned this Jun 13, 2024
Copy link
Contributor

@tonidero tonidero left a comment

Choose a reason for hiding this comment

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

Ups yeah, this is good. Thanks!

@JayShortway JayShortway merged commit 7538a89 into main Jun 14, 2024
8 checks passed
@JayShortway JayShortway deleted the remove-api-key branch June 14, 2024 08:29
tonidero pushed a commit that referenced this pull request Jul 4, 2024
**This is an automatic release.**

### New Features
* Add support for `DEFERRED` upgrades and updates
purchases-hybrid-common to 11.1.0 (#1108) via RevenueCat Git Bot
(@RCGitBot)
* [Android
7.12.0](https://github.com/RevenueCat/purchases-android/releases/tag/7.12.0)
* [iOS
4.43.2](https://github.com/RevenueCat/purchases-ios/releases/tag/4.43.2)
* [iOS
4.43.1](https://github.com/RevenueCat/purchases-ios/releases/tag/4.43.1)
### Dependency Updates
* [AUTOMATIC BUMP] Updates purchases-hybrid-common to 11.0.0 (#1106) via
RevenueCat Git Bot (@RCGitBot)
* [Android
7.12.0](https://github.com/RevenueCat/purchases-android/releases/tag/7.12.0)
* [iOS
4.43.2](https://github.com/RevenueCat/purchases-ios/releases/tag/4.43.2)
* [iOS
4.43.1](https://github.com/RevenueCat/purchases-ios/releases/tag/4.43.1)
* Bump fastlane from 2.220.0 to 2.221.1 (#1102) via dependabot[bot]
(@dependabot[bot])
### Other Changes
* Adds .aiexclude (#1104) via JayShortway (@JayShortway)
* get_phc_version now also checks PHC UI versions (#1100) via
JayShortway (@JayShortway)
* Removes api key from integration test (#1098) via JayShortway
(@JayShortway)
Jethro87 pushed a commit to Jethro87/purchases-flutter that referenced this pull request Jan 4, 2025
Jethro87 pushed a commit to Jethro87/purchases-flutter that referenced this pull request Jan 4, 2025
**This is an automatic release.**

### New Features
* Add support for `DEFERRED` upgrades and updates
purchases-hybrid-common to 11.1.0 (RevenueCat#1108) via RevenueCat Git Bot
(@RCGitBot)
* [Android
7.12.0](https://github.com/RevenueCat/purchases-android/releases/tag/7.12.0)
* [iOS
4.43.2](https://github.com/RevenueCat/purchases-ios/releases/tag/4.43.2)
* [iOS
4.43.1](https://github.com/RevenueCat/purchases-ios/releases/tag/4.43.1)
### Dependency Updates
* [AUTOMATIC BUMP] Updates purchases-hybrid-common to 11.0.0 (RevenueCat#1106) via
RevenueCat Git Bot (@RCGitBot)
* [Android
7.12.0](https://github.com/RevenueCat/purchases-android/releases/tag/7.12.0)
* [iOS
4.43.2](https://github.com/RevenueCat/purchases-ios/releases/tag/4.43.2)
* [iOS
4.43.1](https://github.com/RevenueCat/purchases-ios/releases/tag/4.43.1)
* Bump fastlane from 2.220.0 to 2.221.1 (RevenueCat#1102) via dependabot[bot]
(@dependabot[bot])
### Other Changes
* Adds .aiexclude (RevenueCat#1104) via JayShortway (@JayShortway)
* get_phc_version now also checks PHC UI versions (RevenueCat#1100) via
JayShortway (@JayShortway)
* Removes api key from integration test (RevenueCat#1098) via JayShortway
(@JayShortway)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants