-
Notifications
You must be signed in to change notification settings - Fork 0
feat: [KAN-18] user registration #6
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR implements user registration and activation functionality for the CeramiCraft user microservice, allowing new users to register with email/password and activate their accounts via email verification codes.
- Add user registration API with password validation and email verification
- Add user activation API with 6-digit verification codes
- Implement email service integration for sending activation codes
Reviewed Changes
Copilot reviewed 19 out of 20 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| server/service/register.go | Core registration service with user creation and activation logic |
| server/service/register_test.go | Comprehensive unit tests for registration and activation flows |
| server/http/api/register.go | HTTP API endpoints for registration and activation |
| server/http/router/router.go | Router configuration with custom password validation |
| server/proxy/email.go | Email service implementation for sending activation codes |
| server/repository/dao/user_activation_dao.go | Data access layer for user activation records |
| server/http/data/user.go | Request/response data structures with validation |
| server/config/config.go | Configuration updates for email service |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
b63d4df to
8478844
Compare
8478844 to
666ed51
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 19 out of 20 changed files in this pull request and generated 4 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
7ce4aa7 to
3dd3346
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 19 out of 20 changed files in this pull request and generated 6 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
3dd3346 to
d5213e5
Compare
d5213e5 to
a5f6a11
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 19 out of 20 changed files in this pull request and generated 2 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
923b5b2 to
5dcf837
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 19 out of 20 changed files in this pull request and generated 4 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
5dcf837 to
2adacb6
Compare
name: Pull Request | 合并请求
about: user registration and activation
labels: ["feature"]
变更内容 | Changes
相关 Issue | Related Issue
KAN-18 User Registration
变更类型 | Type of Change
检查清单 | Checklist
变更影响 | Impact
none
测试说明 | Test Instructions
其他说明 | Additional Notes