WordPress.org

Make WordPress Core

Opened 17 months ago

Last modified 5 days ago

#44264 new enhancement

Give progress indication for export and erasure

Reported by: allendav Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version: 4.9.6
Component: Privacy Keywords: needs-design has-patch needs-refresh
Focuses: ui, javascript Cc:
PR Number:

Description

Sometimes export and erasure can take a while, e.g. more than 15 seconds even, on slow hosts or when there is a lot to export.

It would be helpful to somehow indicate to the user how export is proceeding. Since the javascript knows what exporter index it is on, we could at least include that as a percentage or something.

Attachments (2)

44264.0 (4.5 KB) - added by dominic_ks 6 days ago.
Initial patch to show percentage completion of export or deletion.
44264.1.patch (5.1 KB) - added by dominic_ks 5 days ago.
Updated patch with fixed indents and also allows for specific progress indicators per job when running multiple exports or deletions at the same time.

Download all attachments as: .zip

Change History (10)

#1 @desrosj
15 months ago

  • Keywords gdpr removed

Removing the GDPR keyword. This has been replaced by the new Privacy component and privacy focuses in Trac.

#2 @garrett-eclipse
4 weeks ago

  • Focuses ui javascript added
  • Keywords good-first-bug added
  • Milestone changed from Awaiting Review to Future Release

This ticket was mentioned in Slack in #core by dominic_ks. View the logs.


10 days ago

#4 @dominic_ks
10 days ago

Hello, I'm new to contributing but picked this up from the Good First Bugs list and started looking at it this morning.

@dominic_ks
6 days ago

Initial patch to show percentage completion of export or deletion.

#5 @dominic_ks
6 days ago

I've made a couple of updates to the markup of the export / delete in progress button to contain the percentage progress and also the JS to update this element based on the current exporter or eraser index as a percentage of the total.

I found that if you have the "normal" items registered you largely will see 0%, 33%, 67%, done as it works through posts and comments. Since at present there's no indication of the number of pages that will need to be worked through per item.

There's probably a couple of ways that we could find out the total pages, but perhaps that would be overkill.

#6 @rukkykf
5 days ago

  • Keywords has-patch added; needs-patch removed

#7 @rukkykf
5 days ago

  • Keywords needs-testing added

#8 @garrett-eclipse
5 days ago

  • Keywords needs-refresh added; good-first-bug needs-testing removed

Speaking with @dominic_ks on Slack there's a few issues with the initial patch he's working to address, updating to indicate his refresh and remove the call to testing until it's in a better state.

@dominic_ks
5 days ago

Updated patch with fixed indents and also allows for specific progress indicators per job when running multiple exports or deletions at the same time.

Note: See TracTickets for help on using tickets.