- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Advanced Search
Jun 17 2022
Jun 14 2022
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.
Jun 9 2022
Should be better now.
Jun 8 2022
Jun 3 2022
Addressed missing "." and hideDialog() via refs.
Jun 1 2022
May 31 2022
Rebased on master
Test error responses handling
Addressed comments
May 24 2022
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.
May 20 2022
May 18 2022
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.