Page MenuHomePhabricator

Improve Diffusion behavior for no-longer-existing commits
ClosedPublic

Authored by epriestley on Oct 2 2015, 11:04 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 25, 12:59 AM
Unknown Object (File)
Fri, Apr 19, 3:42 PM
Unknown Object (File)
Tue, Apr 16, 8:56 PM
Unknown Object (File)
Fri, Apr 12, 8:30 AM
Unknown Object (File)
Thu, Apr 4, 10:47 AM
Unknown Object (File)
Mar 21 2024, 5:03 AM
Unknown Object (File)
Mar 16 2024, 2:26 AM
Unknown Object (File)
Mar 4 2024, 4:37 PM
Subscribers
None

Details

Summary

Ref T9028. When users push a commit, then later delete it (e.g., by deleting the branch which contained it) we currently explode when trying to view it.

Instead, degrade gradually if some information is not available.

Test Plan
  • Looked at valid commits with parents, refs, branches and merges.
  • Looked at invalid commits.
  • Looked at a previously valid, now-deleted + gc'd commit:

Screen Shot 2015-10-02 at 4.01.27 PM.png (1×1 px, 219 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Improve Diffusion behavior for no-longer-existing commits.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Oct 2 2015, 11:09 PM
This revision was automatically updated to reflect the committed changes.