Page MenuHomePhabricator

Fix create links when no repositories exist
ClosedPublic

Authored by chad on May 5 2016, 3:56 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 24, 10:41 PM
Unknown Object (File)
Sun, Apr 21, 12:50 PM
Unknown Object (File)
Fri, Mar 29, 12:55 AM
Unknown Object (File)
Mar 12 2024, 6:27 AM
Unknown Object (File)
Mar 10 2024, 9:37 AM
Unknown Object (File)
Mar 9 2024, 2:04 PM
Unknown Object (File)
Feb 16 2024, 8:39 AM
Unknown Object (File)
Feb 12 2024, 10:54 AM

Details

Summary

Fixes T10925. Sends users to /new/ instead of /create/

Test Plan

Visit page, see links, clicky clicky.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad retitled this revision from to Fix create links when no repositories exist.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
thoughtpolice added a reviewer: thoughtpolice.
thoughtpolice added a subscriber: thoughtpolice.

This doesn't quite fix it. /diffusion/new has this:

pasted_file (174×915 px, 32 KB)

But clicking on either of those just takes you back to /diffusion/create/ and /diffusion/import which bail out.

I actually have my own commit that fixes it (literally about to say arc diff when I saw this and tested it) but it just changes the path to /diffusion/edit/form/default/ which seems correct, as that is what the top-level Create Repository button on the right does.

This revision now requires changes to proceed.May 5 2016, 4:02 AM

literally, your patch is identical to mine, except those characters

spooky

chad edited edge metadata.
  • Remove /new/ as well
This revision is now accepted and ready to land.May 5 2016, 4:09 AM

This probably can be better from a blank state, but will wait until the dust settles. Thanks for catching the click through issue. I think /new/ gets nuked in all of this, but @epriestley can fix that in the am if not.

This revision was automatically updated to reflect the committed changes.

I'll note the /new/ stuff on T10923 -- it should be getting cleaned up, I just missed it in my cutover.