- We felt the code wasn’t very DRY, probably because you could have used some more abstractions
- You left in namespaces that were blank or commented out
- You had the same ID for everything in the product list. We found this confusing.
- Lots of (nth item 1) and nested (get)s made things a bit hard to follow.
- Various formatting issues made this feel rushed/unfinished.
Hope that is helpful to you.
If you're curious, the code for my project is hosted here.