Differential D17884

[chronik.e.cash] enable trailing slashes for consistent URL structure
ClosedPublic
Actions

Authored by alitayin on Thu, Apr 3, 06:49.

Tags

None

Subscribers

Details

Reviewers

emack
Fabien

Group Reviewers

Restricted Owners Package	(Owns No Changed Paths)
Restricted Project

Commits

rABC431239ff5459: [chronik.e.cash] enable trailing slashes for consistent URL structure

Summary

Due to nginx redirects, URLs without the "/" suffix will lose "docs" in the domain. If we can use trailingSlash to keep in-page routes
ending with slashes, it would largely solve the redirection issues caused by copying and pasting links and refreshing.

Test Plan

Rebuilt the chronik.e.cash documentation in a new environment to test trailingSlash

Diff Detail

Repository

rABC Bitcoin ABC

Branch

alitayin1

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 32878
Build 65241: Build Diff
Build 65240: arc lint + arc unit

Event Timeline

alitayin created this revision.Thu, Apr 3, 06:49

Owners added a reviewer: Restricted Owners Package.Thu, Apr 3, 06:49

Herald added a reviewer: Restricted Project. · View Herald TranscriptThu, Apr 3, 06:49

alitayin requested review of this revision.Thu, Apr 3, 06:49

Harbormaster completed remote builds in B32878: Diff 53348.Thu, Apr 3, 06:49

Trailing slash now added when navigating to various pages

image.png (137×515 px, 13 KB)

This revision is now accepted and ready to land.Thu, Apr 3, 07:25

Fabien accepted this revision.Thu, Apr 3, 07:30

Closed by commit rABC431239ff5459: [chronik.e.cash] enable trailing slashes for consistent URL structure (authored by alitayin). · Explain WhyThu, Apr 3, 07:38

This revision was automatically updated to reflect the committed changes.

alitayin added a commit: rABC431239ff5459: [chronik.e.cash] enable trailing slashes for consistent URL structure.

Revision Contents
Changeset List

Path

Size

web/

chronik.e.cash/

docusaurus.config.js

1 line

Diff 53348

web/chronik.e.cash/docusaurus.config.js

Loading...