Skip to content

develop: solution - #189

Open
ansnihur wants to merge 4 commits into
mate-academy:masterfrom
ansnihur:develop
Open

ansnihur wants to merge 4 commits into
mate-academy:masterfrom
ansnihur:develop

Conversation

@ansnihur

Copy link
Copy Markdown

@2pasha 2pasha left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

good job! 👏

here are some improvements:

  1. when user enter wrong value, there is no ability to fix this field
Image
  1. please fix flow after form submitting, because right now it's 405. You may just reset form or add some popup
Image

@ansnihur
ansnihur requested a review from 2pasha September 15, 2026 12:09

@Zoolgrand Zoolgrand left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Well done, approved

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.

3 participants