search

anschelburk / simple_planner

Apache License 2.0
0 stars 0 forks source link

Code Review #1a: Editable HTMX Boxes #15

Closed anschelburk closed 4 months ago

anschelburk commented 4 months ago

Pull Request 1/2.

Following up on Pull Request #15, I also wanted to see if you can give me any guidance on what I might be doing wrong here. (I followed this guidance from ChatGPT, but it doesn't seem to work, and I'm not sure why.)

I tried to follow this guidance from ChatGPT, but am 98% sure that I messed something up in this code. (In fact, I'm pretty confident that I made many mistakes!) I'm feeling a little stuck, and would love to talk about what I did wrong, and what I can learn for next time.

bbelderbos commented 4 months ago

HTMX != JS ;) it should be doable following the complete htmx flow, we can look at this later together

  1. I think it would be good to work through some of the examples here https://htmx.org/examples/ - we also have some code clinics on the topic
  2. I am thinking this about how to go about this feature: https://chatgpt.com/share/c697b132-f144-4d85-8ce8-644ab6f02c21 -- not sure if we get this done in 30 min, but it offers a good start at least, you see it's quite different from your approach here right?
anschelburk commented 4 months ago

Thanks for the advice. Will work on this further. Closing for now.