Skip to content

Form dev1#1

Open
wojtekfla wants to merge 4 commits into
mainfrom
form-dev1
Open

Form dev1#1
wojtekfla wants to merge 4 commits into
mainfrom
form-dev1

Conversation

@wojtekfla
Copy link
Copy Markdown
Owner

No description provided.

Copy link
Copy Markdown

@RafalKielbasa RafalKielbasa left a comment

Choose a reason for hiding this comment

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

image
According to the guidelines, the form should also accept the PNG format. It would be useful to include information about which formats the form accepts. The initial validation should be performed during the input configuration.

import { FormResume } from "./components/FormResume/FormResume";

function App() {
const [isFormSubmitted, setIsFormSubmitted] = useState(false);
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

In my opinion, this form is unnecessary because this information is stored in the formData state.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The code seems to be oddly formatted. Please verify the functionality of Prettier.

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