Skip to content

[CC-1085] Mark Giropay payment type as deprecated. - #187

Merged
Ryouzanpaku merged 1 commit into
mainfrom
CC-1085/deprecate-giropay
Feb 11, 2026
Merged

[CC-1085] Mark Giropay payment type as deprecated.#187
Ryouzanpaku merged 1 commit into
mainfrom
CC-1085/deprecate-giropay

Conversation

@Ryouzanpaku

Copy link
Copy Markdown
Contributor

No description provided.

@Ryouzanpaku
Ryouzanpaku force-pushed the CC-1085/deprecate-giropay branch from 9602c23 to a589a63 Compare February 11, 2026 08:40
@Ryouzanpaku
Ryouzanpaku force-pushed the CC-1085/deprecate-giropay branch from a589a63 to 01b614a Compare February 11, 2026 09:52
@Ryouzanpaku
Ryouzanpaku merged commit c2dae55 into main Feb 11, 2026
2 of 5 checks passed
@Ryouzanpaku
Ryouzanpaku deleted the CC-1085/deprecate-giropay branch February 11, 2026 10:03
sixer1182 added a commit that referenced this pull request Sep 9, 2026
SofortTest.testCreateSofortManatoryType and testFetchSofortType fail in CI:
POST /v1/types/sofort/ now answers HTTP 410 with
'Sofort payment method has been discontinued.', so the two remaining
enabled tests in the class can never pass again. The third test was
already @disabled('does not work on PAPI').

Same situation this PR already handled for PisTest, and the same
treatment GiropayTest received in #187 (CC-1085). Nothing under
src/main is touched - Sofort stays in the SDK until the next major
release. The two other Sofort references, ChargeTest.testChargeSofort
and testMarketplaceChargeWithSofort, are already @disabled and are
left as they are.

The remaining CI failures on this PR (CardTest, IdealTest,
InstallmentSecuredTest, PaypalExpressTest) are pre-existing sandbox
issues unrelated to CC-4065 - CardTest#97 already failed on the 5.9.0
release PR in February.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011GTcoUbc5ziRDHgWV3hgzQ
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants