airqo-platform / airqo-platform/AirQo-api

Add size limits for bulk updates in notification preferences

未关闭
#4,208 0 条评论 0 个 reaction 已指派 1 人 已被 @Baalmart 认领 在 GitHub 查看
主要语言
JavaScript
星标
26
派生
24
平均合并
5 小时 36 分钟
30 天内合并 PR
81

描述

## Context
Currently, there is no limit on the number of preferences that can be updated in a single bulk update request. This could potentially lead to performance issues with large payloads.

## Proposed Enhancement
Add a maximum limit to the number of preferences that can be updated in a single request to prevent potential performance issues.

## Implementation Details
- Add validation in the bulk update endpoint to limit the number of preferences to 100 per request
- Return appropriate error message when the limit is exceeded

## Related Links
- Pull Request: https://github.com/airqo-platform/AirQo-api/pull/4204
- Discussion: https://github.com/airqo-platform/AirQo-api/pull/4204#discussion_r1921239201

## Additional Notes
This enhancement will help prevent potential performance issues and ensure the API remains stable under load.

/cc @Baalmart

贡献指南

打开贡献指南

评估

这个 Issue 还没有评估数据。

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。