Page MenuHomePhabricator

Namespace Diviner book URLs
AbandonedPublic

Authored by joshuaspence on May 29 2015, 2:44 AM.
Tags
None
Referenced Files
F13186215: D13064.diff
Sat, May 11, 3:29 AM
Unknown Object (File)
Tue, May 7, 6:18 AM
Unknown Object (File)
Fri, May 3, 3:59 AM
Unknown Object (File)
Mon, Apr 29, 4:11 PM
Unknown Object (File)
Wed, Apr 24, 11:56 PM
Unknown Object (File)
Apr 11 2024, 8:13 AM
Unknown Object (File)
Mar 27 2024, 4:20 PM
Unknown Object (File)
Mar 14 2024, 11:50 AM
Subscribers

Details

Summary

Ref T4558. This fixes an inconsistency with Diviner URLs. In DivinerBookPHIDType, handle URIs are set to /diviner/book/{$name}/, but the correct URL is actually /book/{$name}/. Instead of changing DivinerBookPHIDType, I figured that the /book/ routes should probably be nested under the /diviner/ route, as this is consistent with other application routes.

Test Plan

Regenerated documentation with diviner generate and browsed it through the web UI.

Diff Detail

Repository
rP Phabricator
Branch
master
Lint
Lint Warnings
SeverityLocationCodeMessage
WarningREADME.md:23TXT3Line Too Long
WarningREADME.md:27TXT3Line Too Long
WarningREADME.md:39TXT3Line Too Long
Unit
No Test Coverage
Build Status
Buildable 6360
Build 6382: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

joshuaspence retitled this revision from to Namespace Diviner book URLs.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
joshuaspence edited edge metadata.

Found another URL to fix

epriestley edited edge metadata.

It's intentional that /book/ is at top level, similar to /tag/ (and, to a lesser degree, the monogram URLs).

This revision now requires changes to proceed.May 30 2015, 2:01 PM