-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Export returns module #198
base: main
Are you sure you want to change the base?
Conversation
Hi! I'm VTEX IO CI/CD Bot and I'll be helping you to publish your app! 🤖 Please select which version do you want to release:
And then you just need to merge your PR when you are ready! There is no need to create a release commit/tag.
|
Beep boop 🤖 I noticed you didn't make any changes at the
In order to keep track, I'll create an issue if you decide now is not a good time
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome job @Git-the-Sanz. Not sure you have discussed that with @cesarocampov, but I think we need to have a solution to bring the refundData
information. With the current report, the customers cannot have the whole picture if the request is already approved or refunded. Bringing that information can be complex, but I think it's important to discuss it. Maybe we should have a different report for it.
On another note, I remember that we discussed to bring the refundStatusData
in different columns for each status. I see we have a column for new
, but it's only bringing NA
. Lastly, any idea on how to bring the comments?
react/admin/ExportReturnRequestsModal/components/ReportStatus.tsx
Outdated
Show resolved
Hide resolved
Thanks @filafb! I don't understand what you mean with |
The refund data is the object that controls the information regarding what will be refunded for the store user. It has the invoice information, the values that will be refunded but also the items that were approved. That is why it can be complicated to bring the info in the same report. But let's chat about it. And maybe it's better to wait this chat before working on the |
db630a5
to
cd4d746
Compare
cd4d746
to
82f2842
Compare
What problem is this solving?
The ability to export all or some return requests into a single downloadable file
How to test it?
Powerplanet
YAMAMAY
Screenshots or example usage: