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

Account Managment #29

Merged
merged 16 commits into from
Feb 26, 2024
Merged

Account Managment #29

merged 16 commits into from
Feb 26, 2024

Conversation

AnhLe-Axel
Copy link
Collaborator

Main changes:

  • Implement signup, login functionality.
  • Add functionality to modify user's fullName.

Initial development
…ry database

These changes establish the groundwork for login functionality by introducing a basic implementation utilizing a temporary in-memory database. Note that these alterations are provisional and primarily intended as the foundation for further development.
- Add class-transformer and class-validator packages.
- auth.dto.ts: Data Transfer Object for authentication, which defines the shape of the request body.
- main.ts: Binding ValidationPipe at the application level.
- Some properties of user.entity are omitted. These will be update later.
- Move the code for TypeOrmModule from the app.module to its own module called database.module
- This current update function only allows user to change their fullName.
- Still need to create function for each filed in user profile.
@AnhLe-Axel AnhLe-Axel added the Account Management Part of the Account Management user stories of the project label Feb 23, 2024
@AnhLe-Axel AnhLe-Axel added this to the sprint 2 milestone Feb 23, 2024
@AnhLe-Axel AnhLe-Axel self-assigned this Feb 23, 2024
Copy link
Owner

@maingockien01 maingockien01 left a comment

Choose a reason for hiding this comment

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

Feel free to discuss. I may not know everything so correct me with whatever you believe.

src/database/database.module.ts Outdated Show resolved Hide resolved
apps/backend/src/app.module.ts Show resolved Hide resolved
apps/backend/src/users/users.service.ts Outdated Show resolved Hide resolved
apps/backend/src/profile/dto/update.dto.ts Outdated Show resolved Hide resolved
apps/backend/src/users/users.service.ts Outdated Show resolved Hide resolved
apps/backend/src/users/users.service.ts Outdated Show resolved Hide resolved
apps/backend/src/auth/auth.service.ts Outdated Show resolved Hide resolved
apps/backend/src/auth/auth.service.ts Outdated Show resolved Hide resolved
@AnhLe-Axel AnhLe-Axel merged commit 2a6e163 into main Feb 26, 2024
3 checks passed
@maingockien01 maingockien01 added epic user story and removed epic user story labels Apr 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Account Management Part of the Account Management user stories of the project
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants