The share table right now does not represent a share in the sense of T583818 atm, and seems more like an ACL.
It links a 'sharable' to a user meaning it will require an entry for every combination. The idea of the "share" intermediary instead was that a "shareable" could be attached to a share and users could be granted a level of access to the "share", so you would need one extra acl entry to grant access to N sharables, instead of N extra entries, at the cost of an extra indirection.
In the room case, as it is currently use I suppose you'd end up with a share per room, so it's a corner case of the concept.

In favor of D3614

The patch(es) don't apply so I didn't test it or look at the UI. But it sounds ok, and the code looks reasonable so go for it.

I looked into python3 support on centos 8, but I believe we'll have to update the OBS first to get support for modular support.
I'll look into that over the coming weeks and will then look into getting everything up to pykolab python3 ready and packaged.

Should be better now.

This has been merged

Addressed missing "." and hideDialog() via refs.

Rebased on master

Test error responses handling

Addressed comments

Please note: While we could extend this to accept other cryptocurrencies, it currently assumes bitcoin only (and I configured coinbase to only accept bitcoin).
Accepting multiple currencies with a single "coinbase payment method" is feasible, but we currently assume a single currency per method.
In any case, I consider bitcoin only good enough for now, we can extend once we see a need for it.

Please note: While we could extend this to accept other cryptocurrencies, it currently assumes bitcoin only (and I configured coinbase to only accept bitcoin).
Accepting multiple currencies with a single "coinbase payment method" is feasible, but we currently assume a single currency per method.
In any case, I consider bitcoin only good enough for now, we can extend once we see a need for it.

Looks reasonable, but I haven't tried to apply since the previous patches didn't apply.

Looks good to me but doesn't currently apply.

This patch doesn't apply on top of master with the other patches as specified in the ticket.