I update Recurly module to 7.x-2.0-alpha1 and use Recurly.JS to build subscription page, but I see that we don't have CVV input text field for this update.

Could anyone help me on this?

CommentFileSizeAuthor
#10 cvv_input_field_missing-2680929-10.patch1.04 KBmarkdorison
#4 where_is_cvv_input-2680929-4.patch1.04 KBmarkdorison

Comments

nvhcuong created an issue. See original summary.

Rob_Feature’s picture

Rob_Feature commented

Yeah, I'm seeing the same thing...that was a red flag not to use this 2.x version of recurlyjs forms yet.

Rob_Feature’s picture

Rob_Feature commented
Category: Support request » Bug report
Priority: Normal » Major

Marking as major and a bug since I can't imagine why this would be excluded on purpose. Seems recurly.js signup forms would be non-functional without this field.

markdorison’s picture

markdorison
he/him
commented
Version: 7.x-2.0-alpha1 » 7.x-2.x-dev
Status: Active » Needs review
StatusFileSize
new where_is_cvv_input-2680929-4.patch1.04 KB

I've added the CVV field to the billing forms but I am having trouble verifying it in testing. Recurly's test gateway doesn't seem to have a way to test the CVV values.

mmatsoo’s picture

mmatsoo commented
Status: Needs review » Reviewed & tested by the community

+1 The patch applied nicely and the CVV field appeared on the sign-up form, as expected.

NB! I did not do a live transaction. I only confirmed that patch applies and that the CVV field appears on the form.

  • markdorison committed 8572ac9 on 7.x-2.x 
    Issue #2680929 by markdorison: Where is CVV input?
markdorison’s picture

markdorison
he/him
commented
Title: Where is CVV input? » CVV input field missing.
Status: Reviewed & tested by the community » Fixed
markdorison’s picture

markdorison
he/him
commented
Version: 7.x-2.x-dev » 8.x-1.x-dev
Status: Fixed » Patch (to be ported)
markdorison’s picture

markdorison
he/him
commented
Title: CVV input field missing. » CVV input field missing from billing form.
markdorison’s picture

markdorison
he/him
commented
Status: Patch (to be ported) » Needs review
StatusFileSize
new cvv_input_field_missing-2680929-10.patch1.04 KB
mmatsoo’s picture

mmatsoo commented
Status: Needs review » Reviewed & tested by the community

D8 version of patch applied successfully and the CVV field appeared.

  • markdorison committed 02db016 on 8.x-1.x 
    Issue #2680929 by markdorison: CVV input field missing from billing form
markdorison’s picture

markdorison
he/him
commented
Status: Reviewed & tested by the community » Fixed
Rob_Feature’s picture

Rob_Feature commented

Just to ask the obvious question: Has someone completed a successful transaction after this patch was put in? Just making sure there's a little thing called "It works" that has been confirmed since no one mentioned that. :)

markdorison’s picture

markdorison
he/him
commented

@Rob_Feature I have only done so on test accounts, not on a live charge. I'd also love to hear other feedback.

nvhcuong’s picture

nvhcuong commented

@Rob_Feature and @markdorison
I tested live charge and this issue fixed. I got successful paid transaction.

Thanks for your update markdorison!

markdorison’s picture

markdorison
he/him
commented

@nvhcuong Thank you for testing! Was that on D8 or D7?

nvhcuong’s picture

nvhcuong commented

I am working on D7.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.