WordPress.org

Make WordPress Core

Opened 19 months ago

Last modified 2 months 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 2 months ago.
Initial patch to show percentage completion of export or deletion.
44264.1.patch (5.1 KB) - added by dominic_ks 2 months 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
17 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
3 months 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.


2 months ago

#4 @dominic_ks
2 months 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
2 months ago

Initial patch to show percentage completion of export or deletion.

#5 @dominic_ks
2 months 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
2 months ago

  • Keywords has-patch added; needs-patch removed

#7 @rukkykf
2 months ago

  • Keywords needs-testing added

#8 @garrett-eclipse
2 months 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
2 months 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.