absmach / magistrala

Industrial IoT Messaging and Device Management Platform
https://www.abstractmachines.fr/magistrala.html
Apache License 2.0
2.46k stars 667 forks source link

Improve listing and searching requests #2326

Open dborovcanin opened 2 months ago

dborovcanin commented 2 months ago
          We are treating IDs differently, i.e. taking it into account only if other criteria is not there, making `buildQueryConditions` mask the logic of processing IDs. Why?

_Originally posted by @dborovcanin in https://github.com/absmach/magistrala/pull/2305#discussion_r1663881189_

dborovcanin commented 2 months ago

We are using the same DTO struct to pass data between search and list functions, and it's becoming too cumbersome because we often don't need all data from the DTO, beating the purpose of the DTO itself.