serokell / coffer

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

[#119] Refactor `set-field` in CLI and WebAPI #130

Open sancho20021 opened 2 years ago

sancho20021 commented 2 years ago

Description

Problem: currently set-field is used both for changing visibility and setting new fields. This results in additional handling of the situation where both field contents and visibility option are missing.

Solution: require contents for setting field and allow changing visibility for convenience. Add set-field-visibility command in CLI and route in WebAPI which modifies only visibility.

Related issue(s)

Fixes #119

:white_check_mark: Checklist for your Pull Request

Related changes (conditional)

Stylistic guide (mandatory)

sancho20021 commented 2 years ago

We've added new cmd. Seems we have to add it to README

True. Fixed. And also documentation about status codes