Skip to content

Conversation

@justinh24
Copy link

Store and Unstore methods were already implemented
Passing the 'stored token' to purchase would return an error.

Implimented 'ProcessCreditCardWithCustomerCodeV1'
Added Unit Test
Added Remote Test

Store and Unstore methods were already implimented
Passing the Stored token to purchase would return an error.

Finished implimenting 'ProcessCreditCardWithCustomerCodeV1'
Added Unit Test
Added Remote Test
add_description(post, options)

commit((payment.is_a?(Check) ? :purchase_check : :purchase), post)
commit((payment.is_a?(Check) ? :purchase_check : payment.is_a?(String) ? :purchase_token : :purchase), post)
Copy link
Contributor

Choose a reason for hiding this comment

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

Nested ternary operators 🙅‍♂️ could you please change this to something similar to the structure used in add_payment?

@bpollack bpollack added the gateway/feature Adds a new feature to an existing gateway label Jun 25, 2018
@bpollack
Copy link
Contributor

@bdewater Is this something you're still interested in landing?

@github-actions
Copy link

To provide a cleaner slate for the maintenance of the library, this PR/Issue is being labeled stale after 60 days without activity. It will be closed in 14 days unless you comment with an update regarding its applicability to the current build. Thank you!

@github-actions github-actions bot added the Stale label Oct 24, 2025
@github-actions github-actions bot closed this Nov 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gateway/feature Adds a new feature to an existing gateway Stale

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants