decentralized-identity / decentralized-identity/dwn-sql-store

Refactor `processFilter` and `processTags`

Open
#29 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
TypeScript
Stars
7
Forks
6
PR merge metrics
No merged PRs in 30d

Description

`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

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.