consul

Commit Graph

Author	SHA1	Message	Date
John Cowen	b256313256	ui: [BUGFIX] Ensure in-folder KVs are created in the correct folder (#10569 ) When clicking to create a KV within folder name, would would be viewing a form that was a form for creating a KV in the root, which when the user clicked to save, saved the KV in the root. For the moment at least I've removed the code that strips double slashes, and whilst this isn't ideal, it looks like we've picked up one of those bugs that turns into a 'feature', and completely reworking KV to not rely on the double slashes is not really an option right now.	2021-07-14 18:49:01 +01:00
John Cowen	6589cbbd0d	ui: Move to Workspaced Structure (#8994 ) * ui: Add the most basic workspace root in /ui * We already have a LICENSE file in the repository root * Change directory path in build scripts ui-v2 -> ui * Make yarn install flags configurable from elsewhere * Minimal workspace root makefile * Call the new docker specific target * Update yarn in the docker build image * Reconfigure the netlify target and move to the higher makefile * Move ui-v2 -> ui/packages/consul-ui * Change repo root to refleect new folder structure * Temporarily don't hoist consul-api-double * Fixup CI configuration * Fixup lint errors * Fixup Netlify target	2020-10-21 15:23:16 +01:00

Author

SHA1

Message

Date

John Cowen

b256313256

ui: [BUGFIX] Ensure in-folder KVs are created in the correct folder (#10569 )

When clicking to create a KV within folder name, would would be viewing a form that was a form for creating a KV in the root, which when the user clicked to save, saved the KV in the root.

For the moment at least I've removed the code that strips double slashes, and whilst this isn't ideal, it looks like we've picked up one of those bugs that turns into a 'feature', and completely reworking KV to not rely on the double slashes is not really an option right now.

2021-07-14 18:49:01 +01:00

John Cowen

6589cbbd0d

ui: Move to Workspaced Structure (#8994 )

* ui: Add the most basic workspace root in /ui

* We already have a LICENSE file in the repository root

* Change directory path in build scripts ui-v2 -> ui

* Make yarn install flags configurable from elsewhere

* Minimal workspace root makefile

* Call the new docker specific target

* Update yarn in the docker build image

* Reconfigure the netlify target and move to the higher makefile

* Move ui-v2 -> ui/packages/consul-ui

* Change repo root to refleect new folder structure

* Temporarily don't hoist consul-api-double

* Fixup CI configuration

* Fixup lint errors

* Fixup Netlify target

2020-10-21 15:23:16 +01:00

2 Commits (0eb4a98fabc4f1d6ec73875ad3b0881a86d17f9f)