Features update version and update noti Change Password #15
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…d detailed results
…with hashed passwords from backup
- Updated SlaveNode interface to include sync configuration and metrics. - Implemented database migration for slave nodes and sync logs. - Created API endpoints for registering, updating, deleting, and syncing slave nodes. - Added middleware for validating slave API keys. - Developed service layer for handling slave node operations. - Implemented query options for fetching slave nodes and their statuses. - Enhanced mock data for slave nodes to include new fields.
- Introduced SystemConfig interface in types for managing node configurations. - Created migration for system_configs table with necessary fields and enum for node modes. - Implemented system configuration controller with endpoints to get, update, connect, disconnect, and test connection to master node. - Added routes for system configuration management. - Developed service methods for API interactions related to system configuration. - Implemented query options for fetching system configuration with automatic refetching.
…nd ACLs for clarity; remove unused URL validation utility
…rvice checks, dependency updates, and health checks
… before generating Prisma client and running migrations; streamline logging
…rectly and enhance logging for Prisma client generation, migrations, and seeding
…te script; streamline backend and frontend build processes
…from Prisma schema
…mproved logging; update update script steps
There was a problem hiding this comment.
Pull Request Overview
This pull request implements a comprehensive features update including version management, notification system updates, and password change functionality improvements. The changes introduce master-slave node synchronization, backup/restore capabilities, and enhanced UI components.
Key Changes:
- Added master-slave node synchronization system with API key authentication
- Implemented backup scheduling and configuration import/export functionality
- Enhanced user account management with improved password change notifications
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| scripts/update.sh | Adds automated update script for rebuilding and restarting services |
| apps/web/src/types/index.ts | Extended SlaveNode interface with sync configuration and metrics |
| apps/web/src/services/*.ts | New services for system config, slave nodes, and backup management |
| apps/web/src/components/pages/*.tsx | Updated SlaveNodes and Backup pages with full functionality |
| apps/api/src/controllers/*.ts | New controllers for backup, slave node, and sync management |
| apps/api/prisma/schema.prisma | Database schema updates for backup and slave node features |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Comment on lines
+32
to
37
| import { toast } from "sonner"; | ||
| import { accountService } from "@/services/auth.service"; | ||
|
|
||
| const Account = () => { | ||
| const { t } = useTranslation(); | ||
| const { toast } = useToast(); | ||
|
|
There was a problem hiding this comment.
Inconsistent toast library usage - the file imports both toast from 'sonner' and removes the useToast hook, but other parts of the codebase still use the useToast pattern. This creates inconsistency across the application.
|
vncloudsco
added a commit
that referenced
this pull request
Oct 8, 2025
* Feat: Update Features Backup & Restore (#12) * feat: Update Features Backup & Restore * Feat: Slave Mode (#13) * feat: Update Features Backup & Restore * Feat: Features update (#14) * feat: Update software * Features update version and update noti Change Password (#15) * feat: Update Features Backup & Restore * Fix frontend error (#16) * Refactor services to use centralized API module and token storage * Feat: Enhance Slave Mode UI with mode switch button and update node mode mutation * feat: Improve SSLDialog layout with enhanced text wrapping for certificate fields * refactor: replace js-cookie with localStorage for token management (#17) * feat: add syncInterval and lastSyncHash columns to system_configs table (#18) * feat: Update project goal description and remove security recommendation * feat: Update project goal description and remove security recommendation * About readme (#21) * feat: Update project goal description and remove security recommendation * Refactor be (#22) * Refactor code structure for improved readability and maintainability * style: limit max height of certificate, private key, and chain input fields (#23) --------- Co-authored-by: SangND <dacsang97@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.