GitLab

Person type in CIM is optional

See merge request !22

This influences what the persons can do inside CIM users.  If not set,
CIM uses default value configured on the server.

Make parent_key optional again

See merge request !20

Use None instead of empty string to say the field is empty

CIM-58 Rull ut og klargjør til akseptansetest

See merge request !19

* Upsert: Return the response

* Delete: Return an i tuple of an enum describing whether person(s)
were deleted or if none was found and the response.

* Log error and return raw response
* Fix type annotation on import_organisations

The API endpoint is called upsert

Use set comprehension instead of map()

* Fixes CIM-59, CIM-60 and CIM-61

* Partial object updates are only partially supported.  Change logic
  to always serialise whole object, but exclude empty phone numbers.

* Remove unused code

* Validate models on assignment

* Simplify types.  Remove constr fields, mypy don't understand them and
  CIM validates data anyway.

* Use mixins to support optional functionality

Field names in CIM are already lower_snake_case

We don't use this functionality

Token removal

See merge request !18

Instead of throwing json decoding errors when getting responses with
error codes and text, we let the call method raise appropriate errors
before trying to do the json decoding. Results in less code and easier
error handling.

Having a separate token dict in addition to the handlers one makes
things unnecessarily complex. If you want to add an api key you can
simply add it as any other header.

Since /webservices is always part of the url it's actually part of the
base and should be handled as such.

Use correct argument

See merge request !17

Minor fixes

See merge request !16

Update docs

See merge request !15

Feature/add linters and type checking

See merge request !14

They are created during function definition time. All calls to the
function reuse this one instance of that data structure, persisting
changes between them.