Skip to content

feat: add user profile management with validation (#306)#523

Merged
nanaf6203-bit merged 2 commits into
MettaChain:mainfrom
Peolite001:feature/306-user-profile-management
May 29, 2026
Merged

feat: add user profile management with validation (#306)#523
nanaf6203-bit merged 2 commits into
MettaChain:mainfrom
Peolite001:feature/306-user-profile-management

Conversation

@Peolite001
Copy link
Copy Markdown
Contributor

Closes #306

##Changes Made

Created UpdateProfileDto with comprehensive validation rules (email format, phone number, URL, timezone, contact hours HH:MM format, etc.)
Created ProfileResponseDto for consistent, enriched profile responses including computed fullName and statistics
Enhanced UsersService with getProfile() and updateProfile() methods
getProfile() returns enriched data with property/transaction counts and account age
updateProfile() validates unique email, builds partial updates, logs activity to ActivityLog
Updated UsersController endpoints: GET /users/me/profile and PUT /users/me/profile

@drips-wave
Copy link
Copy Markdown

drips-wave Bot commented May 27, 2026

@Peolite001 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

- Add UpdateProfileDto with comprehensive input validation
- Add ProfileResponseDto for structured profile responses
- Implement getProfile and updateProfile in UsersService
- Add activity logging for profile updates
- Update /users/me/profile GET and PUT endpoints
@Peolite001 Peolite001 force-pushed the feature/306-user-profile-management branch from f11e3bf to 8d834b9 Compare May 27, 2026 10:31
@nanaf6203-bit
Copy link
Copy Markdown
Contributor

@Peolite001 fix the CI and resolve conflicts.

@nanaf6203-bit
Copy link
Copy Markdown
Contributor

@Peolite001

@Peolite001
Copy link
Copy Markdown
Contributor Author

@Peolite001

ok

Copy link
Copy Markdown
Contributor

@nanaf6203-bit nanaf6203-bit left a comment

Choose a reason for hiding this comment

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

LGTM

@nanaf6203-bit nanaf6203-bit merged commit c2ee1d3 into MettaChain:main May 29, 2026
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 User Profile Management

2 participants