TBD54566975 / dwn-sql-store

Apache License 2.0
8 stars 6 forks source link

Refactor `processFilter` and `processTags` #29

Open LiranCohen opened 6 months ago

LiranCohen commented 6 months ago

processFilter and processTags have similar functionality and could likely benefit from a slight refactor.

The key difference between the two is that the filters will always match the property's column name, but the tags match against a different column (valueNumber or valueString) depending on the filter value's type.

https://github.com/TBD54566975/dwn-sql-store/blob/main/src/utils/filter.ts#L13-L69