From 77d664efe7d7d78561f4f0fee20b3fae0099ee75 Mon Sep 17 00:00:00 2001 From: BonfaceKilz Date: Tue, 19 Oct 2021 13:32:09 +0300 Subject: Update notes on authorisation --- issues/authorisation.gmi | 23 +++++++++++++++++++++-- 1 file changed, 21 insertions(+), 2 deletions(-) diff --git a/issues/authorisation.gmi b/issues/authorisation.gmi index ffca299..4d3f924 100644 --- a/issues/authorisation.gmi +++ b/issues/authorisation.gmi @@ -1,6 +1,10 @@ # Clean up Authorisation -Look at Chris' authorisation code[0]. Atm genenetwork2's handling of +Look at Chris' authorisation code: + +=> https://github.com/chfi/gn-proxy + +Atm genenetwork2's handling of authorisation is a bit clunky; Since @Bonface is working on the data data up-loader, and it requires authorisation, this is a good time to clean up authorisation. @@ -50,5 +54,20 @@ On the UI(quote from Zach): > have people direct input e-mails (instead of giving the ability to > search). +#### Mon 18 Oct 2021 + +- Marked the function `check_owner_or_admin` as deprecated. It's only + used in: `wqflask/show_trait/show_trait.py`. + +- Heavy use of the walrus operator to remove alot of boiler plate. + +#### Tue 19 Oct 2021 + +- Every resource needs to have the 3 fields set for the group-mask: + "data", "metadata", "admin". Otherwise, a hash-ref error will be + thrown: -[0] https://github.com/chfi/gn-proxy +``` +03 ~~> hash-ref: no value found for key + key: 'admins +``` -- cgit v1.2.3