Open-event-frontend: Add a proper button for Pay Now to pay for the tickets through stripe

Created on 4 Aug 2018  路  22Comments  路  Source: fossasia/open-event-frontend

To Reproduce
Steps to reproduce the behavior:

  1. Go to e/event_identifier then order tickets
  2. Click on order
  3. Scroll down and fill info and choose stripe as payment option.
  4. See error

Expected behavior

Add a proper UI to the Pay now button.

Desktop (please complete the following information):

  • OS: Ubuntu
  • Browser: Chromium
UUX newbie-friendly

Most helpful comment

@Anupam-dagar Correct url will be /e/{event_identifier} where event identifier is the id of the event created by you. Then to access the orders page make sure you have enabled the payment option and have proper keys set up for paypal. You should see the paynow button for paypal then.

All 22 comments

@ritikamotwani claim

@ritikamotwani I couldnt find the page e/event_identifinder?

@ritikamotwani I am claiming this and working on it.

@ritikamotwani I would like to work on this

@ritikamotwani what is the correct url, I am getting a 404 on localhost/e/event_identifier

@Anupam-dagar Correct url will be /e/{event_identifier} where event identifier is the id of the event created by you. Then to access the orders page make sure you have enabled the payment option and have proper keys set up for paypal. You should see the paynow button for paypal then.

@ritikamotwani @pradeepgangwar I am claiming this.

@srv-twry For this I have to modify stripe selector ui or pay no button

@sk9331657 you have to modify the Pay Now button.

screenshot from 2018-12-26 00-53-55
This one

@ritikamotwani Am I referring right button??

@ritikamotwani @pradeepgangwar I've made required styling change. Please verify if this style is fine?
Basically, I just added the class of Pay Now button of Order form to the stripe checkout Pay with card button.
If you both are fine with the style, I'll proceed to open a PR for this issue. Do suggest me if any kind of style change is required :))
screenshot from 2019-01-03 18-04-25

@ritikamotwani @pradeepgangwar I've made required styling change. Please verify if this style is fine?
Basically, I just added the class of Pay Now button of Order form to the stripe checkout Pay with card button.
If you both are fine with the style, I'll proceed to open a PR for this issue. Do suggest me if any kind of style change is required :))
screenshot from 2019-01-03 18-04-25

@shreyanshdwivedi Please ask maintainers before taking up the issue. I have already claimed the issue.

@sk9331657 it has been 10 days since you claimed the issue. And as no progress was made in 10 days by you, I think anyone was free to take up this issue. 10 days are sufficiently enough time to solve this :sweat_smile:

@shreyanshdwivedi @sk9331657 Sorry for late reply. But I would like to inform you that both of you are referring to wrong button.

Actually the button that we are talking of comes when user selects stripe as the payment option and proceeds next.

@pradeepgangwar I think there is a bit of confusion here. Please confirm which button you are referring to -
_Step 1 -_
screenshot_2019-01-04 new order -40db264b-69d2-42c6-8716-6fd3091085b6 orders open event

_Step 2 -_
screenshot_2019-01-04 placed order -40db264b-69d2-42c6-8716-6fd3091085b6 orders open event

_Step 3 -_
screenshot from 2019-01-04 16-24-59

@shreyanshdwivedi as said earlier the step after we choose to pay via stripe I.e. we are referring to button in step 2.

@pradeepgangwar I've opened the PR for button in Step 2 itself :wink:
Please refer to my PR https://github.com/fossasia/open-event-frontend/pull/1781

@shreyanshdwivedi actually we need original button that is provided via stripe. Please read the stripe documentation and see if you can attach that button as we get for paypal.

@pradeepgangwar Okay. I'll try to embed the button provided by Stripe. :smile:

Cool. If that is not possible please let me know we can proceed with this for now.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

prateekj117 picture prateekj117  路  5Comments

mariobehling picture mariobehling  路  5Comments

dilpreetsio picture dilpreetsio  路  4Comments

mariobehling picture mariobehling  路  3Comments

mariobehling picture mariobehling  路  3Comments