feat: Allow edition of published charts #1208

bprusinowski · 2023-10-10T13:39:33Z

This PR allows the edition of published charts.

~~Configurator state key is now stored inside the config~~
During publishing, we check if user is logged in and if a chart of given id exists in the database; if so, we update the chart config, assuring that they can only update the charts they created (client side and API side guards)
New ?edit=configuratorStateKey initializes chart state with configuratorStateKey = edit mode
Old ?from=configuratorStateKey initializes chart state with a new chart id = new chart mode

vercel · 2023-10-10T13:39:39Z

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
visualization-tool	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Oct 11, 2023 7:27am

app/components/chart-selection-tabs.tsx

bprusinowski added 4 commits October 10, 2023 13:28

refactor: Add key to configurator state

e62a823

refactor: Extract useUser hook

c06a350

feat: Allow edition of published charts

8df2923

fix: Add missing properties

7addeb4

bprusinowski requested a review from ptbrowne as a code owner October 10, 2023 13:39

vercel bot deployed to Preview October 10, 2023 13:42 View deployment

refactor: Do not store configurator key in config

e52034d

vercel bot deployed to Preview October 10, 2023 14:04 View deployment

feat: Add indication that user is editing published chart

7a6ac8f

vercel bot deployed to Preview October 10, 2023 14:34 View deployment

ptbrowne reviewed Oct 10, 2023

View reviewed changes

app/components/chart-selection-tabs.tsx Outdated Show resolved Hide resolved

refactor: useFetchData

8318ef1

vercel bot deployed to Preview October 11, 2023 07:27 View deployment

ptbrowne approved these changes Oct 11, 2023

View reviewed changes

bprusinowski merged commit ecb7add into main Oct 11, 2023
3 of 4 checks passed

bprusinowski deleted the feat/allow-editing-charts-when-logged-in branch October 11, 2023 08:44

Provide feedback