Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/User validation #20

Merged
merged 5 commits into from
Sep 18, 2024
Merged

Conversation

GSzabi11
Copy link
Collaborator

No description provided.

@GSzabi11 GSzabi11 linked an issue May 28, 2024 that may be closed by this pull request
4 tasks
Copy link
Member

@csiszaralex csiszaralex left a comment

Choose a reason for hiding this comment

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

You have the same problems like #19 please check it and resolve the problems

package.json Outdated Show resolved Hide resolved
@balintking balintking self-requested a review August 28, 2024 23:41
Copy link
Collaborator

@balintking balintking left a comment

Choose a reason for hiding this comment

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

Sorry for the little late, please resolve these few things quickly and we can close this issue

apps/backend/src/user/entities/user.entity.ts Outdated Show resolved Hide resolved
apps/backend/src/user/dto/create-user.dto.ts Outdated Show resolved Hide resolved
@GSzabi11 GSzabi11 requested a review from balintking September 15, 2024 11:51
Copy link
Member

@csiszaralex csiszaralex left a comment

Choose a reason for hiding this comment

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

Nice work!

@csiszaralex csiszaralex merged commit b310b4f into main Sep 18, 2024
2 checks passed
@csiszaralex csiszaralex deleted the feature/16-user-validation-dtos branch September 18, 2024 18:13
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.

Add Class Validation to User Entity
3 participants