-
Notifications
You must be signed in to change notification settings - Fork 0
Implement statistics module with activity tracking and user statistics #9
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
…cation; update method signatures and repository queries accordingly.
…eamDto with consolidated FileSystemStreamRequestDto hierarchy
…ove user data structure handling, added "stats" microservice to the CI / CD workflow
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 pull request introduces a statistics module with activity tracking and implements a comprehensive user identification system transition from usernames/numeric IDs to UUIDs. Additionally, it adds dark mode support to the frontend and updates chart versions to stable releases.
- UUID-based user identification system throughout authentication and core services
- New statistics service with activity tracking via RabbitMQ streams
- Dark mode implementation in the frontend with user settings management
Reviewed Changes
Copilot reviewed 103 out of 105 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| helm/Encryptify-Stats/* | Helm chart configuration for new statistics service |
| encryptify-stats/* | Complete statistics service implementation with activity tracking |
| encryptify-core/* | Updated to use UUID user identification and activity stream integration |
| encryptify-auth/* | Migrated from username-based to UUID-based authentication system |
| encryptify-frontend/* | Added dark mode support and enhanced user settings management |
| helm/*/Chart.yaml | Updated chart versions from SNAPSHOT to stable releases |
Files not reviewed (2)
- .idea/compiler.xml: Language not supported
- .idea/runConfigurations/Run_Application.xml: Language not supported
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
This pull request introduces a significant refactor to the user identification system, transitioning from using usernames and numeric IDs to universally unique identifiers (UUIDs) throughout the authentication and user management services. The changes impact entity models, repositories, authentication flows, and event handling, ensuring greater consistency and scalability in user identification.
Core changes to user identification:
EncryptifyUserentity now uses aUUIDas its primary key instead of aLong, and all related repository interfaces and service methods have been updated to useUUIDfor user identification. [1] [2]JwtAuthenticationProvider,UserDetailsImpl,UserPrincipal) have been updated to useUUIDfor user IDs instead of usernames or numeric IDs, including token subject extraction and principal name representation. [1] [2] [3] [4]Service and controller updates:
UUIDinstead of usernames, including account deletion, fetching user info, and checking if a user is banned. Controllers have been updated accordingly to use the new API. [1] [2] [3] [4] [5]Event and communication changes:
UserReadyForSetupEventand email verification processes. [1] [2] [3] [4]Build and configuration updates:
encryptify-authwas updated from0.0.1-SNAPSHOTto0.0.1..idea/compiler.xmlwas updated to include theencryptify-statsmodule and to rename an annotation profile. [1] [2]These changes collectively improve the robustness and future-proofing of user identification across the authentication system.