DDC

Results 27 comments of DDC

Thanks for these comments @chriskmnds I am working through them. > and if there are any concerns with storing the actual GridPlans there, then I'd still like us to use...

> Honestly, this feels complex to me. There is a lot going on here, which I'd prefer we revise the approach at a higher level first, before diving into the...

Thanks for the feedback Christos, I have addressed all of them :) To summarise: 1 - The single source of truth is now hook state all store related code is...

Thanks for the comments @chriskmnds I am going to close this PR now since I have to shift to other priorities and the code from this is mostly unnecessary. Will...

@chriskmnds I was going to close it because when I come back I might have to most probably face merge hell 😁 . Far easier to just create a new...

@chriskmnds turns out I do have enough time so I did the changes that you requested. Please do have a look when you have some time.

@chriskmnds I am planning to merge this. Let me know if you have any objections.

Self note: Set default state as null so that we have an indication of what fields the user interacts with.

@southp sorry I had to short circuit the PR to allow Rita to see the form as you mentioned! A feature flag is a better alternative for this to force...

> If we just need a place to load the experiment on entering a flow, we do have a simple callback now here The problem with this is that experiment...