David Cook

Results 169 comments of David Cook

Awesome, thanks Filipe! I've created an issue for this: - https://github.com/openfoodfoundation/openfoodnetwork/issues/11845 I think we shouldn't delete the legacy spec until we remove the feature toggle.

Hi Filipe, I've updated the `tomselect_*select` methods in this PR to be more flexible: https://github.com/openfoodfoundation/openfoodnetwork/pull/12183/commits/520482b8b69ca5e5de618235bce80df9bbabd998 Once it's merged, you'll have the option of using the label text (just like `fill_in`),...

> **Option A:** Fully remove SKU on product level > > * Remove the input field for SKU on page `/products`. > * Search the code for places where the...

Thank you! This solved it for me. I wish I could contribute the fix, but can't figure out how. I wonder if this affects all other stimulus-components.

I wondered if we can set this up with Ansible? Even better, it should probably be an `.env` variable, which both openfoodnetwork and newrelic could refer to.

I just checked one example from a deployment for a single server. It took 5 mins, but the first three mins were setup, and it only took two minutes after...

Sorry I didn't get to this today, but as discussed @heroinedor may help be able to help today. Please ping me again if I should jump in!

Thank you Luis, this is really helpful feedback! I've responded to some comments and will take this feedback on when I pick the task back up next year.

User Interface: (some tidying up to be done..) ![Screen Shot 2023-02-07 at 3 29 32 pm](https://user-images.githubusercontent.com/4188088/217148677-61629328-cf69-4055-b1e6-af248bf1514b.png) ![Screen Shot 2023-02-07 at 3 29 43 pm](https://user-images.githubusercontent.com/4188088/217148686-bafe40c6-56bd-4694-8cec-2e43e7bd6096.png)

Thanks @luisramos0 , this is valuable feedback. It seems this system design is not quite right, but I'm not clear on the best path and would like to communicate with...