Skip to content

Conversation

@ugochukwu-stripe
Copy link

@ugochukwu-stripe ugochukwu-stripe commented Mar 21, 2025

Summary

Add Create Server Side switch to the dev test application, when enabled it’ll create payment intents using the stipe api, and when disabled it’ll create payment intents via the terminal.createPaymentIntent. Server side create is always enabled for p400 readers as it doesn’t support client side creation. Also made a similar change for set up intents.

Motivation

So testing create PI/SI -> retrieve PI/SI -> collect path does not require code changes.

Testing

  • I tested this manually
  • I added automated tests

Documentation

Select one:

  • I have added relevant documentation for my changes.
  • This PR does not result in any developer-facing changes.
    Screenshot 2025-03-21 at 7 39 24 PM
    Screenshot 2025-04-01 at 12 09 10 PM

@ugochukwu-stripe ugochukwu-stripe changed the title Add Client side create button to the dev app Add Server side create button to the dev app Apr 1, 2025
@nazli-stripe
Copy link
Collaborator

@ugochukwu-stripe this is causing errors for example build for iOS. Can you rebase off of main and see if you can resolve them?

Copy link
Collaborator

@nazli-stripe nazli-stripe left a comment

Choose a reason for hiding this comment

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

build errors for iOS example build

When enabled it’ll create payment intents using the terminal sdk, and when disabled it’ll create  payment intents via the stripe api. Client side create is always disabled for p400 readers as it doesn’t support that functionality.

Signed-off-by: Ugo Chukwu <[email protected]>
Rename enable client side -> enable server side, enabled by default for Verifone P400.

Signed-off-by: Ugo Chukwu <[email protected]>
@ugochukwu-stripe ugochukwu-stripe force-pushed the add-server-side-create-option-to-test-app branch from 0e9e1f0 to d703491 Compare April 8, 2025 21:26
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