Don't let Diffusion show that an importing repository is "100%" imported
ClosedPublic
Actions

Authored by epriestley on Jan 24 2014, 7:05 PM.

Tags

None

Referenced Files

	F14022907: D8058.diff
	Wed, Nov 6, 8:58 PM

	Unknown Object (File)
	Oct 7 2024, 1:24 PM

	Unknown Object (File)
	Oct 7 2024, 1:24 PM

	Unknown Object (File)
	Oct 7 2024, 1:24 PM

	Unknown Object (File)
	Oct 7 2024, 1:13 PM

	Unknown Object (File)
	Oct 2 2024, 11:04 AM

	Unknown Object (File)
	Sep 21 2024, 9:43 AM

	Unknown Object (File)
	Sep 20 2024, 5:41 AM

View All Files

Subscribers

aran

Details

Reviewers

btrahan

Commits

Restricted Diffusion Commit
rP53687827c643: Don't let Diffusion show that an importing repository is "100%" imported

Summary

A few users have hit this and found it confusing. Currently, it means "more than 99.95%", which is very different from "100%". Instead:

show an extra digit of precision; and
cap the display at "99.99%", so it's more clear that work is still happening.

Test Plan

Faked it and saw it cap at 99.99%.

Diff Detail

Branch

percents

Lint

Lint Passed

Unit

No Test Coverage

Event Timeline

btrahan accepted this revision.Jan 24 2014, 7:24 PM

Closed by commit rP53687827c643 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

diffusion/

controller/

DiffusionRepositoryEditMainController.php

8 lines

Diff 18233

View Options

src/applications/diffusion/controller/DiffusionRepositoryEditMainController.php

Don't let Diffusion show that an importing repository is "100%" importedClosedPublicActions