ccfos / nightingale

An all-in-one observability solution which aims to combine the advantages of Prometheus and Grafana. It manages alert rules and visualizes metrics, logs, traces in a beautiful web UI.

https://flashcat.cloud/docs/

Apache License 2.0

9.55k stars 1.39k forks source link

feat: Provide optional style for buildTargetWhere #2038

Closed chenyunda218 closed 1 month ago

chenyunda218 commented 1 month ago

What type of PR is this? Enhancement What this PR does / why we need it: models.buildTargetWhere 每當參數增加減少都需要改動。

2034 可令這個PR用另一種優雅的方式實現。

710leo commented 1 month ago

这个建议挺好，可以提一个完整的 PR ：）

chenyunda218 commented 1 month ago

會希望保留buildTargetWhere增加一個buildTargetWhereWithOptions還是直接取代原本的buildTargetWhere？我比較偏向後者。

710leo commented 1 month ago

buildTargetWhere

可以直接取代原来的 buildTargetWhere

chenyunda218 commented 1 month ago

感覺Limit, Offset, Order 可以做成optional，但目前先這樣？

710leo commented 1 month ago

感覺Limit, Offset, Order 可以做成optional，但目前先這樣？

可以先这样