Getting a TypeError when calling Payment.objects.create() about drf-instamojo HOT 6 CLOSED

@sumit4613 We will have to fix this piece of code. Looks like instamojo library has been updated or either this was not ever run. You are point-blank on the RCA. Can you raise a PR for this fix?

Also, let's start building test cases on this.

from drf-instamojo.

@Nirpesh87 According to the traceback here, the reason you're getting TypeError is because payment_request_imojo.get('modified_at') here returns str and not datetime.datetime. And you can't compare different types. Also, I'm not sure why it returns str, if you can show me your code which is causing this maybe I can help in this issue.

Please let us know the steps to reproduce this. I'll try to reproduce it myself.

Thanks.

from drf-instamojo.

@Nirpesh87 According to the traceback here, the reason you're getting TypeError is because payment_request_imojo.get('modified_at') here returns str and not datetime.datetime. And you can't compare different types. Also, I'm not sure why it returns str, if you can show me your code which is causing this maybe I can help in this issue.

Please let us know the steps to reproduce this. I'll try to reproduce it myself.

Thanks.

@sumit4613 , should i create a different view for PaymentSerializer to save payment record?

from drf-instamojo.

@Nirpesh87 According to the traceback that was shared by you. The problem isn't in View or Serializer. There's some issue in signal which was called when saving Payment object. I'll have to look why payment_request_imojo.get('modified_at') is returning str rather than datetime.datetime. As @iamhssingh suggested maybe instamojo have updated their library or there's some issue in this code. I'll have to debug this and will raise a PR for this issue.

If you can fix this issue, please let us know or raise a PR with your changes.

Thanks!

from drf-instamojo.

Waiting for this bug to be fixed :(

from drf-instamojo.

@Nirpesh87 We're working on the issue, we'll create a new release in a day or two. So please be patient.

from drf-instamojo.