Skip to content
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

Include File Download Logs in Data Export requests #6499

Closed
cklosowski opened this issue Apr 10, 2018 · 6 comments
Closed

Include File Download Logs in Data Export requests #6499

cklosowski opened this issue Apr 10, 2018 · 6 comments
Assignees
Milestone

Comments

@cklosowski
Copy link
Contributor

Enhancement Request

File Download logs contain IP addresses, email addresses and customer name. When Exporting customer data, these should be included.
screen shot 2018-04-10 at 12 34 02 pm

screen shot 2018-04-10 at 12 35 30 pm

Justification or use case

This is a GDPR issue.

@SeanTOSCD
Copy link
Contributor

I think something may be wrong with the file download export data. I'm testing with a customer that has 4 file download logs. 3 of the logs are for one file on a purchase and the other log is for a file on another purchase. See the logs here:

screenshot 2018-05-16 12 58 12

Notice the timestamps for the 4 logs. Now here's the export data:

screenshot 2018-05-16 12 59 32

It looks like all Date of Download values adopt the latest file download log timestamp. Even before downloading the file from the second purchase, meaning the export was showing 3 file downloads from the same purchase, it was still using the Date of Download from the last of the 3 file downloads for all of them.

So the export is working but and all the other info looks good, but Date of Download appears to have an issue.

@mintplugins
Copy link
Contributor

I am having the same issue with the date as @SDavisMedia - everything else looks good though!

cklosowski added a commit that referenced this issue May 18, 2018
@cklosowski
Copy link
Contributor Author

@SDavisMedia @mintplugins Ok date should be fixed.

@SeanTOSCD
Copy link
Contributor

Unfortunately I can't test this right now because there's some sort of error during the email sending/data downloading. Both just hang and I'm getting this error in debug bar:

TypeError: focusedRowActions is undefinedhttp://localhost/sandhills/wp-admin/js/common.js?ver=4.9.6 line 623

This is happening on WP nightly as well as 4.9.6. Not sure what's going on but I can't get to my export.

@cklosowski
Copy link
Contributor Author

@SDavisMedia Just updated this to fix any merge conflicts as wel.

cklosowski added a commit that referenced this issue May 21, 2018
@SeanTOSCD
Copy link
Contributor

Okay this is looking good for me now! 👍

screenshot 2018-05-21 12 54 55

cklosowski added a commit that referenced this issue May 21, 2018
* Adding initial file download log export #6499

* Correct the array keys #6499

* Fix log date #6499
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants