B2Bitcoin / beBOP

Marvelous p2p bitcoin-based online sales platform
https://be-bop.io
GNU Affero General Public License v3.0
15 stars 3 forks source link

On /admin/config, add option to hide either email or nostr options #1449 #1452

Closed ithiame closed 2 days ago

Tirodem commented 1 week ago

With both methods : image With only Nostr and empty field : image Please add a front control on input to avoid that. Note that, in the case both email + nostr are enabled, if mail is filled but nostr is not, no error should be triggered because of empty nostr address : it must only happen when nostr is the only displayed field.

Tirodem commented 1 week ago

It's OK for nostr only ✅ BTW, if I set to email only, nostr is now always displayed. As we hesitated, we decided to allow nostr disabling. So, removing nostr should remove the nostr field. And while testing it, I saw we're able to do this : image Currently, it'll display nostr input. But, for now, it's better to oblige be-BOP owner to have at least either email, or nostr, or both, but not none. In the future we'll try do to no-contact options for anon be-BOP, but as it's now, we need one or another. And nostr should be masked if the owner wants, like email.