Radish alpha
h
rad:z3gqcJUoA1n9HaHKufZs5FCSGazv5
Radicle Heartwood Protocol & Stack
Radicle
Git
cli: improve rad id allow list UX
Merged fintohaps opened 1 year ago

The previous implementation of using the --allow option in rad id update would be an absolute list, rather than additive. That is, if there were existing DIDs not passed in via --allow then they would be remove.

Change this behaviour to check the existing value of the document’s allow list, and add to it if --allow is specified.

To enable the user to also remove a DID, the --disallow option is also added.

The correct usage of these options is improved by checking the validity of the --visibility option being used, the current state of the repository’s privacy, and the use of the --allow and --disallow options.

fintohaps opened with revision ed97c94f on base 064ece32 +216 -9 1 year ago

The previous implementation of using the --allow option in rad id update would be an absolute list, rather than additive. That is, if there were existing DIDs not passed in via --allow then they would be remove.

Change this behaviour to check the existing value of the document’s allow list, and add to it if --allow is specified.

To enable the user to also remove a DID, the --disallow option is also added.

The correct usage of these options is improved by checking the validity of the --visibility option being used, the current state of the repository’s privacy, and the use of the --allow and --disallow options.

fintohaps pushed revision 2 f91d9297 on base 6f91f2fb +206 -9 1 year ago

Changes:

  • Error if disallow and allow are not disjoint
  • Improve test to show off use of multiple --allow and --disallow
did:key:z6MksFqX...wzpT pushed revision 3 8ef265ea on base 6f91f2fb +206 -9 1 year ago

Small CLI output edits.

did:key:z6MksFqX...wzpT merged revision 8ef265ea at f26be552 1 year ago