Skip to content

Fix duplicated docs hash links#282

Merged
krowvin merged 1 commit into
mainfrom
273-some-links-still-broken
Jun 1, 2026
Merged

Fix duplicated docs hash links#282
krowvin merged 1 commit into
mainfrom
273-some-links-still-broken

Conversation

@krowvin

@krowvin krowvin commented May 11, 2026

Copy link
Copy Markdown
Contributor

Summary

Fixes

Release impact

  • Fixes to URL routing and navigation. No breaking changes to current GW api.

@krowvin krowvin requested a review from willbreitkreutz May 11, 2026 21:21
@krowvin krowvin linked an issue May 11, 2026 that may be closed by this pull request
@github-actions

Copy link
Copy Markdown

Please set a versioning label of either major-bump, minor-bump, or patch-bump to the pull request.

@willbreitkreutz

Copy link
Copy Markdown
Contributor

Toss out the last commit as duplicative with the changes already merged.

@krowvin krowvin force-pushed the 273-some-links-still-broken branch from 9eb5b97 to 88653af Compare May 19, 2026 02:27
@krowvin krowvin merged commit c37e3bf into main Jun 1, 2026
@krowvin krowvin deleted the 273-some-links-still-broken branch June 1, 2026 17:27
willbreitkreutz pushed a commit that referenced this pull request Jun 2, 2026
willbreitkreutz added a commit that referenced this pull request Jun 8, 2026
* initial commit for maps

* styles update

* fixed old type

* fixed library name

* fixed conflicts

* Update publish.yml (#276)

* run npm audit fix (#275)

* 4.0.6

* Update publish.yml (#277)

* 4.0.7

* try updated checkout action (#279)

* 4.0.8

* Update publish (#280)

* try updated checkout action

* add repository to package.json?

* 4.0.9

* Update publish.yml

try and get the npm logs

* Update publish.yml

* Update publish.yml

Remove debug step

* 278 corps castle link not redirecting correctly (#281)

* Run at /groundwork locally to better emulate docs site

* Modify docs template to make home button and breadcrumbs work better.  Remove any logic from breadcrumb items, just use the href provided.

* Add a `homeUrl` prop to the site wrapper that drives the href of the usace logo in the header, just pass that href down, or if not provided bail with a `#`

* add `homeUrl` prop to SiteWrapper docs

* 4.1.0

* Fix duplicated docs hash links (#282)

* 4.1.1

* try and get up to date package lock

* update groundwork-geo

* remove annoying type errors

* change from console.log to alert, that way we show the user click handler is doable

* swap alert for console.log

* make mapping navigation links consistent

* Fix mapping section of links in nav-links.js

* add demo imports for the components being used

* fix demo import for GeoJSONLayer

* Update map-layout docs

Note that it is leftSidebar and the other overlays are ...Toolbar

---------

Co-authored-by: thill02 <tessa.j.hill@usace.army.mil>
Co-authored-by: Charles Graham <charles.r.graham@usace.army.mil>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Some links still broken

2 participants