Feature/map#222
Conversation
|
@tracyfraley1 Here's the work @thill02 did for the mapping. Will is going to take a look. Then see if we can get it merged in. |
|
any status update on this? |
| "dependencies": { | ||
| "@headlessui/react": "^2.2.0", | ||
| "@tailwindcss/forms": "^0.5.9", | ||
| "@usace/groundwork-geo": "^2.0.4", |
There was a problem hiding this comment.
I attempted a fresh install on local and saw it can load two React versions and crash the docs app.
It was some error along the lines of
ReactCurrentDispatcher and usace_groundwork-geo.js.
I think we need a PR for @usace/groundwork-geo to make react/react-dom peer deps, or for this app to pin/override React so only one version is installed?
| const pageBreadcrumbs = [ | ||
| { | ||
| text: "Documentation", | ||
| href: "/docs", |
There was a problem hiding this comment.
Nav links are not using the apps established hash/base URL pattern.
Existing docs nav links use BASE_URL + "#/...", but the new mapping links use absolute paths like /docs/mapping: src/nav-links.js (line 263).
Need to ensure all links have the BASE_URL prepended
| ({ store }) => { | ||
| const props = f.getProperties(); | ||
|
|
||
| console.log(props); |
3 participants
No description provided.