Do not open tickets if the order status is not completed

[liveHarshit]

Describe the bug Currently, tickets are fetching if the order status is incompleted.

Would you like to work on the issue?

Open for all

[captain82]

@liveHarshit I would like to work on this issue.

[liveHarshit]

@liveHarshit I would like to work on this issue.

Please go ahead. Also, we assign one issue at a time. For reference #1932

[captain82]

@liveHarshit I will work on this issue first.

[captain82]

@liveHarshit Can you describe a little what exactly needs to happen?

[liveHarshit]

@liveHarshit Can you describe a little what exactly needs to happen?

In the tickets section, order status is displaying for tickets. Sometimes an order is pending(due to incomplete payment), then instead of opening ticket details, it should redirect for payment. And for another status (canceled or expire), only a message should display. Open the ticket only when order is completed or placed(in case of offline payment)

[captain82]

@liveHarshit ok Thank you

[captain82]

@liveHarshit How do I make an incomplete payment order?

[liveHarshit]

@liveHarshit How do I make an incomplete payment order?

With order identifier. Check how order is creating in attendee fragment. API docs: https://open-event-api-dev.herokuapp.com

Open Event API Server

[captain82]

@liveHarshit when the status is pending the user should be redirected to Attendee Fragment where the countdown of 15 minutes begins to complete the payment,right?

[liveHarshit]

@liveHarshit when the status is pending the user should be redirected to Attendee Fragment where the countdown of 15 minutes begins to complete the payment,right?

I'm not sure if we can update attendees. Please check the frontend implementation. Debug: https://open-event-fe.netlify.com Release: eventyay.com

[captain82]

Thanks @liveHarshit

[captain82]

@liveHarshit When i try pay using paymal sandbox ,it says your selected payment method was not accepted.Need help.

[liveHarshit]

@liveHarshit When i try pay using paymal sandbox ,it says your selected payment method was not accepted.Need help.

Use client Id: AXrRge-gN1E5WHMYsu0FsftPOWBjTu3Sqj0pKfpq7mPKwv6w_ikiqWrka_06lAWZW5_WxL9CwaMiTVsc with the debug server.

[captain82]

@liveHarshit Pending Tickets are now being paid successfully using paypal .How do i test for Stripe payment.I am unable to create a event with stripe as payment method.

[liveHarshit]

@liveHarshit Pending Tickets are now being paid successfully using paypal .How do i test for Stripe payment.I am unable to create a event with stripe as payment method.

If it is working with PayPal then do the same for stripe.

[captain82]

@liveHarshit I think pending orders can be created only with PayPal and not Stripe as no parameter is obtained for card details. If a user chooses Stripe payment it can either be a success or failure during first-time ticket booking. Correct me if I am wrong?

[liveHarshit]

@liveHarshit I think pending orders can be created only with PayPal and not Stripe as no parameter is obtained for card details. If a user chooses Stripe payment it can either be a success or failure during first-time ticket booking. Correct me if I am wrong?

User needs to enter details again.

[captain82]

@liveHarshit So do i have to make a new screen for that?

[liveHarshit]

user should be redirected to Attendee Fragment

[captain82]

@liveHarshit Are you sure? Because the same could have been done for Paypal

[liveHarshit]

@liveHarshit Are you sure? Because the same could have been done for Paypal

What if the user wants to pay with stripe. Open attendee section with same order identifier. So that order will complete after payment is done. Got it?

[liveHarshit]

@iamareebjamal Your opinion?

[captain82]

@liveHarshit Are you sure? Because the same could have been done for Paypal

What if the user wants to pay with stripe. Open attendee section with same order identifier. So that order will complete after payment is done. Got it?

I think this is possible .Lets see what @iamareebjamal says

[captain82]

@liveHarshit Should i proceed with your mentioned method or wait for @iamareebjamal

[iamareebjamal]

Yes

[captain82]

@liveHarshit I want to make a PR for this bug

[liveHarshit]

@liveHarshit I want to make a PR for this bug

Yes please.

[captain82]

@liveHarshit ok

[captain82]

@liveHarshit Just made a PR.

[kshitij-99]

Please close this issue.

[liveHarshit]

Please close this issue.

Why?

[thatfedupguy]

Is this issue still open ??...would like to work on it.

[rob729]

@liveHarshit Can I work on this issue?

[liveHarshit]

There is already a PR. @captain82 What is the status?

[iamareebjamal]

2 months, so most likely won't continue

[liveHarshit]

@rob729 Go ahead. First read all comments on the previous PR and understand the issue.

[thatfedupguy]

@liveHarshit I have asked first if I can work on this issue or not. Please assign it to me.

[liveHarshit]

@liveHarshit I have asked first if I can work on this issue or not. Please assign it to me.

GO AHEAD

[rob729]

@tusharr-98 Are you still working on this issue?

[thatfedupguy]

@rob729 yes...I have asked some questions related to the last PR from @liveHarshit but haven't got any reply.

[iamareebjamal]

Where's the question?

[thatfedupguy]

https://github.com/fossasia/open-event-attendee-android/pull/2356#discussion_r349884297

[thatfedupguy]

@liveHarshit, how should I get the order and event by their ids as they are required as a parameter to start PayPal payment?

[liveHarshit]

@liveHarshit, how should I get the order and event by their ids as they are required as a parameter to start PayPal payment?

From order fragment and order under user view model.

[thatfedupguy]

I am unable to create a pending order, need help

[thatfedupguy]

Also every time I am trying to create order...snackbar appears with a message "unable to create order".

[rob729]

@tusharr-98 Are you still working on this issue?

[thatfedupguy]

@rob729 are you able to create Order ??

[thatfedupguy]

@liveHarshit help needed

[liveHarshit]

@liveHarshit help needed

yes?