refactor(notification): 移除通知服务中的IP地址参数#338
Conversation
laoshuikaixue
commented
Apr 11, 2026
laoshuikaixue
commented
- 移除createSongSelectedNotification函数中的clientIP参数
- 移除createSongPlayedNotification函数中的clientIP参数
- 移除createSystemNotification函数中的ipAddress参数
- 移除createBatchSystemNotifications函数中的ipAddress参数
- 移除createSongVotedNotification函数中的ipAddress参数
- 移除createSongRejectedNotification函数中的ipAddress参数
- 移除createSubmissionNoteClearedNotification函数中的ipAddress参数
- 移除createReplayRequestRejectedNotification函数中的ipAddress参数
- 移除sendEmailNotificationToUser函数中的ipAddress参数
- 移除sendBatchEmailNotifications函数中的ipAddress参数
- 清理相关调用处的IP地址获取和传递代码
- 移除createSongSelectedNotification函数中的clientIP参数 - 移除createSongPlayedNotification函数中的clientIP参数 - 移除createSystemNotification函数中的ipAddress参数 - 移除createBatchSystemNotifications函数中的ipAddress参数 - 移除createSongVotedNotification函数中的ipAddress参数 - 移除createSongRejectedNotification函数中的ipAddress参数 - 移除createSubmissionNoteClearedNotification函数中的ipAddress参数 - 移除createReplayRequestRejectedNotification函数中的ipAddress参数 - 移除sendEmailNotificationToUser函数中的ipAddress参数 - 移除sendBatchEmailNotifications函数中的ipAddress参数 - 清理相关调用处的IP地址获取和传递代码
|
Deployment failed with the following error: Learn More: https://vercel.com/laoshuikaixues-projects?upgradeToPro=build-rate-limit |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
✅ Deploy Preview for voice-hub-lhzx ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request refactors the notification system by removing the clientIP and ipAddress parameters from multiple service functions and API endpoints. The review identifies unused variables and imports in server/api/admin/schedule.post.ts and server/api/admin/schedule/publish.post.ts that resulted from these changes and suggests their removal to maintain code cleanliness.
| }, | ||
| clientIP | ||
| } | ||
| ) |
There was a problem hiding this comment.
在移除 createSongSelectedNotification 中的 clientIP 参数后，该文件中的 clientIP 变量（第 127 行）已不再被使用。建议将其定义以及相关的 getClientIP 导入（第 6 行）一并移除，以保持代码整洁。
| }, | ||
| clientIP | ||
| } | ||
| ) |
There was a problem hiding this comment.
在移除 createSongSelectedNotification 中的 clientIP 参数后，该文件中的 clientIP 变量（第 96 行）已不再被使用。建议将其定义以及相关的 getClientIP 导入（第 7 行）一并移除，以保持代码整洁。
- 删除 schedule/publish.post.ts 中未使用的 getClientIP 导入 - 删除 schedule/publish.post.ts 中未使用的客户端IP获取代码 - 删除 schedule.post.ts 中未使用的 getClientIP 导入 - 删除 schedule.post.ts 中未使用的客户端IP获取代码
1 participant
