Skip to content

refactor komponentu do tworzenia projektów#64

Merged
mtrznadel24 merged 7 commits into
developfrom
fix-ProjectForm
May 26, 2026
Merged

refactor komponentu do tworzenia projektów#64
mtrznadel24 merged 7 commits into
developfrom
fix-ProjectForm

Conversation

@FAZ110
Copy link
Copy Markdown
Owner

@FAZ110 FAZ110 commented May 25, 2026

No description provided.

Comment thread .claude/settings.local.json Outdated
{
"permissions": {
"allow": [
"Bash(Get-ChildItem -Path \"c:\\\\Users\\\\jandy\\\\OneDrive\\\\Pulpit\\\\6 sem\\\\IO\\\\IO-Project\" -Directory)"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Tego pliku nie powinno być na repo

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

racja XD


<div className="space-y-3">
{milestonesFields.map((field, index) => {
const lastMilestone = milestones[index - 1];
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Przez ułamek sekundy to może być undefined czy coś, chyba bezpieczneij by było tak:
const lastMilestone = milestones?.[index - 1];

}
return next;
}, [usersById, foundUsers]);
const selectedUserIds = useWatch({ control, name: roles }) ?? [];
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Linter chyba dobrze mówi z tym warningiem, możesz to owinąć w useMemo, to załatwi sprawę

})),
});
}).refine(
(data) => !data.startDate || !data.endDate || data.endDate >= data.startDate,
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

patrząc na komunikat błędu to chyba to !data.startDate || !data.endDate nie powinno być sprawdzane tutaj

Comment thread frontend/src/features/project/components/CreateProjectForm.view.tsx Outdated
Comment thread frontend/src/features/project/components/CreateProjectForm.view.tsx Outdated
@mtrznadel24 mtrznadel24 merged commit 44504fc into develop May 26, 2026
2 checks passed
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.

4 participants