accepted
merge request
!51
"Add missing dependency in setup.py"
at
geant-swd / dashboardv3 / inventory-provider
-
bb1ad455 · Merge branch 'patch/dependency' into 'develop'
- ... and 1 more commit. Compare 1327f670...bb1ad455
opened
merge request
!51
"Add missing dependency in setup.py"
at
geant-swd / dashboardv3 / inventory-provider
accepted
merge request
!50
"Dboard3 1142/token auth"
at
geant-swd / dashboardv3 / inventory-provider
-
1327f670 · Merge branch 'DBOARD3-1142/token-auth' into 'develop'
- ... and 7 more commits. Compare d4b283eb...1327f670
-
c564be2c · pep8
-
10f5ba5c · Add type hinting and Enum definition for service names
commented on
merge request !50
"Dboard3 1142/token auth"
at
geant-swd / dashboardv3 / inventory-provider
Added type hinting.
commented on
merge request !50
"Dboard3 1142/token auth"
at
geant-swd / dashboardv3 / inventory-provider
Added type hinting.
commented on
merge request !50
"Dboard3 1142/token auth"
at
geant-swd / dashboardv3 / inventory-provider
Updated.
commented on
merge request !50
"Dboard3 1142/token auth"
at
geant-swd / dashboardv3 / inventory-provider
Updated to use Enum.
-
4259a38f · Add type hinting and Enum definition for service names
commented on
merge request !50
"Dboard3 1142/token auth"
at
geant-swd / dashboardv3 / inventory-provider
type hint this def
commented on
merge request !50
"Dboard3 1142/token auth"
at
geant-swd / dashboardv3 / inventory-provider
if client names are an Enum, type hinting
commented on
merge request !50
"Dboard3 1142/token auth"
at
geant-swd / dashboardv3 / inventory-provider
these DEFAULT_REQUEST_HEADERS* are only used once each. It's cleaner to define them only once where they're used ... this way increases the possib...
commented on
merge request !50
"Dboard3 1142/token auth"
at
geant-swd / dashboardv3 / inventory-provider
I think these would be more elegant as Enums. And then you can add type hinting to authorize().