serokell / coffer

Multi-backend password store with multiple frontends
4 stars 2 forks source link

[#44] Merge `--filter` and `--filter-field` options #94

Closed sancho20021 closed 2 years ago

sancho20021 commented 2 years ago

Description

Problem

@Heimdell in #44 suggested to merge --filter and --filter-field options into a single --filter option.

e.g.:

--filter name~vault               <- filter entries by entry name
--filter url:value~google.com     <- filter entries by field value

Solution

Modified parseFilter :: MParser Filter supplementing it with parseFilterByField's functionality.

Changed related datatypes so there is no separation between filter and filter-field.

Related issue(s)

:white_check_mark: Checklist for your Pull Request

Related changes (conditional)

Stylistic guide (mandatory)