Skip to content

added the mannequin functionality - #149

Open
diana-siwa wants to merge 7 commits into
Lazztech:mainfrom
diana-siwa:main
Open

diana-siwa wants to merge 7 commits into
Lazztech:mainfrom
diana-siwa:main

Conversation

@diana-siwa

Copy link
Copy Markdown

Added a mannequin functionality to position the outfits on a canvas.

@gianlazz

gianlazz commented Oct 7, 2026

Copy link
Copy Markdown
Member

On first skim the implementation looks good. I assume though by the 1,200 lines added in one commit that this was done with a coding agent.

What model did you use?

@gianlazz

gianlazz commented Oct 7, 2026 •

Copy link
Copy Markdown
Member

This looks pretty good too, but it needs some polish before it's releasable.

image

I'm not a fan of the extra settings page linked from the navbar. Instead, let's have the canvas default settings linked from the profile route (http://localhost:3000/auth/profile). That's where I intended to be the root for settings navigation.

image

Also looks like the pipeline is failing due to formatting issues. Please run npm run precommit which will resolve this. I'll also noodle on this and expect that I'll have more feedback.

@gianlazz gianlazz left a comment •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See the comment above 👆

@diana-siwa

Copy link
Copy Markdown
Author

Single commit is because I don't commit in between testing, probably a bad habit. I used chatgpt to help me learn how typescript works, as I'm not very experienced with the syntax. I'll move the settings, shouldn't be an issue.

@diana-siwa

Copy link
Copy Markdown
Author

Oh, okay I see why I added the settings on the navbar, it's because the profile is only visible if auth is enabled (I don't intend to use it personally, but I'll add it to commit).

@diana-siwa
diana-siwa requested a review from gianlazz October 7, 2026 16:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants