-
Notifications
You must be signed in to change notification settings - Fork 254
fix: error 500 lms/dashboard after applying ecommerse coupon #4003
fix: error 500 lms/dashboard after applying ecommerse coupon #4003
Conversation
Thanks for the pull request, @DmytroAlipov! Please note that it may take us up to several weeks or months to complete a review and merge your PR. Feel free to add as much of the following information to the ticket as you can:
All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here. Please let us know once your PR is ready for our review and all tests are green. |
2583b6e
to
c4e8544
Compare
Hi @openedx/revenue-squad! Friendly ping on this! |
Hi @openedx/revenue-squad! Could we get an ETA on getting this reviewed? |
@mphilbrick211 |
@mphilbrick211 @DmytroAlipov I'm looking into it. Will try to approve it asap. |
@DmytroAlipov 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future. |
@zubair-ce07 Thank you very much! |
…upported#4003) Co-authored-by: Muhammad Zubair <syedzubairtahir12@gmail.com>
Description
Steps to Reproduce:
The issue was that the date-time also included milliseconds, which is not acceptable for converting the date and time into the required format.
ValueError: time data '2033-06-25T11:40:12.267999Z' does not match format '%Y-%m-%dT%H:%M:%SZ'