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

feat(admin): add team info sync functionality #213

Merged
merged 3 commits into from
Mar 11, 2025
Merged

feat(admin): add team info sync functionality #213

merged 3 commits into from
Mar 11, 2025

Conversation

aide-cloud
Copy link
Collaborator

  • Add SyncTeamInfo RPC to team.proto for synchronizing team information
  • Implement SyncTeamInfo method in team business logic and repository
  • Add locking mechanism to prevent concurrent synchronization
  • Migrate sync logic from Data initialization to Team repository
  • Update API permissions for team info sync

- Add SyncTeamInfo RPC to team.proto for synchronizing team information
- Implement SyncTeamInfo method in team business logic and repository
- Add locking mechanism to prevent concurrent synchronization
- Migrate sync logic from Data initialization to Team repository
- Update API permissions for team info sync
- Remove syncBizDatabase method from Data struct
- Delete syncBizDatabase call from initDatabase function
- Clean up related code and comments

This change removes unused code related to business database synchronization, simplifying the database initialization process.
- Move resolved alarm deletion to a separate goroutine
- Add error recovery for the deletion process
- Use a copy of the context for the deletion operation
@jinronga jinronga self-requested a review March 11, 2025 09:34
@jinronga jinronga merged commit 6916d50 into main Mar 11, 2025
@aide-cloud aide-cloud deleted the feature/sync branch March 11, 2025 09:42
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.

2 participants