DDC
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...