Skip to content

Move ENVIRONMENT to xcconfig#2248

Closed
atmamont wants to merge 34 commits intodevelopfrom
chore/move-environment-to-xcconfig
Closed

Move ENVIRONMENT to xcconfig#2248
atmamont wants to merge 34 commits intodevelopfrom
chore/move-environment-to-xcconfig

Conversation

@atmamont
Copy link
Copy Markdown
Contributor

@atmamont atmamont commented Sep 5, 2025

Summary [Required]

Demo [Required for UI changes]

Ticket [Optional]

COSDK-585

Todo

  • Replace old PAL endpoint with the new checkout/{version}/delete/storedPaymentMethods/(storedPaymentMethodId) (remove stored payment methods)
  • Remove PAL environment

Checklist [Required]

  • Tested changes locally
  • Tested app release on CI
  • Tested on beta environment
  • Added/updated unit tests
  • Verified against acceptance criteria

atmamont and others added 24 commits August 21, 2025 16:19
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
# Summary
Update AdyenCardScanner podspec

## Checklist [Required]
<!-- Mark completed items with an [x] -->
- [x] Tested changes locally
- [x] Added/updated unit tests
- [x] Verified against acceptance criteria
Add write access to post release job

## Checklist [Required]
<!-- Mark completed items with an [x] -->
- [x] Tested changes locally
- [x] Added/updated unit tests
- [x] Verified against acceptance criteria

---------

Co-authored-by: Naufal Aros <naufal.aros@adyen.com>
@atmamont atmamont added the chore a pull request that has chore changes that shouldn't be in the release notes label Sep 5, 2025
github-actions[bot]
github-actions Bot previously approved these changes Sep 5, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Sep 5, 2025

ℹ️ No baseline data found for 'develop'.

Framework Size
AdyenComponents.framework 2680 KB
AdyenActions.framework 2881 KB
AdyenDropIn.framework 1282 KB
AdyenDelegatedAuthentication.framework 82 KB
AdyenAuthentication.framework 347 KB
AdyenWeChatPay.framework 257 KB
AdyenCardScanner.framework 710 KB
AdyenEncryption.framework 517 KB
AdyenSession.framework 855 KB
AdyenTwint.framework 267 KB
AdyenWeChatPayInternal.framework 4135 KB
AdyenCashAppPay.framework 1771 KB
AdyenCard.framework 1690 KB
TwintSDK.framework 97 KB
Adyen3DS2.framework 923 KB
Adyen.framework 7499 KB
total 25993 KB

github-actions[bot]
github-actions Bot previously approved these changes Oct 16, 2025
github-actions[bot]
github-actions Bot previously approved these changes Oct 16, 2025
@github-actions
Copy link
Copy Markdown
Contributor

✅ No changes detected

Comparing chore/move-environment-to-xcconfig to develop


Analyzed targets: Adyen, AdyenActions, AdyenCard, AdyenCardScanner, AdyenCashAppPay, AdyenComponents, AdyenDelegatedAuthentication, AdyenDropIn, AdyenEncryption, AdyenSession, AdyenSwiftUI, AdyenTwint, AdyenWeChatPay

@atmamont atmamont marked this pull request as ready for review October 17, 2025 14:03
@sonarqubecloud
Copy link
Copy Markdown

@atmamont atmamont requested review from a team, descorp and erenbesel and removed request for a team October 17, 2025 14:19
# componentsEnvironment
sed -i '' "s/static let componentsEnvironment.*=.*/static let componentsEnvironment = Environment.$ENVIRONMENT/g" ./Demo/Configuration.swift
# Environment is now configured via xcconfig and Info.plist
echo "Using ENVIRONMENT=$ENVIRONMENT setting from configuration"
Copy link
Copy Markdown
Contributor

@descorp descorp Oct 17, 2025

Choose a reason for hiding this comment

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

The whole step "⚙️ Configure Demo App" is no longer needed, 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.

This is a very good point, @descorp ! Thanks for catching

@atmamont
Copy link
Copy Markdown
Contributor Author

atmamont commented Feb 9, 2026

Hasn't been picked up for a few months.

@atmamont atmamont closed this Feb 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore a pull request that has chore changes that shouldn't be in the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants