Add linting python and yaml and error checking json files to pipeline

See merge request !31

Failed to update tests on the last commit.

The raising of exceptions when posting batches results in the
Unknown-status never beeing returned from the API. Aditionally, it
exposes transport specifics out of the client module. One should only do
one of them, not both. Exception raising when posting sales orders is
now removed.

The ordenr_ubw attribute is never sent in, only returned by the Setra
API. The model handles both input and output of salesorders, so we'll
have to mark the attribute as optional and 

Forgot about the tests. Should probably test more variants of the sales
order number.

The order number is transferred along with the status of import back to
BookitLab.

Treating 404 differently based on the reasons and multiple fixes

See merge request !30

BOOK-46: Made two batch fields optional, because sometimes when you retrieve a batch...

See merge request !29

it will not fill these field, because they are not yet available, and therefore it would fail when parsing the data 

Setra-133: Bugfix setra client batch api search

See merge request !28

setra was crashing because wrong parameters was used to search for batches

Setra111: uniformere svar fra put og post endepunkter

See merge request !27

Use correct JSON decode exception

See merge request !26

requests.exceptions.JSONDecodeError is non existent in requests versions
before 3.0.0. Therefore requests is pinned to 2.24 or greater, and we
use ValueError instead of requests.exceptions.JSONDecodeError.

Setra-114: Sotra detail model changes: Make product_specification field optional

See merge request !25

Rework the put_update_batch() interface

See merge request !24

Don't send null values when updating batch

See merge request !23

This commit makes the interface of put_update_batch() quit similar to
post_new_batch().

The Setra API does not accept null values for attributes, but it accepts
that the attribute is omitted. We should probably accept null values
for fields in order to avoid obscure bugs like this, but it does not
hurt to omit the values.

At least we use less of the bandwidth! :)

Added sotra models and urls

See merge request !19

Post new batches without sending None-fields

See merge request !21

Exclude unset values from beeing transfered when posting a new batch to
Setra. Setra does not accept fields set to None (at least period and
vouchertype).

Redefine the interface for post_new_batch()

See merge request !20

This interface avoids leaking transport-specific information, while
returning an indication of how the request fared so the user can decide
on how to proceed with processing.

Clarify responses from post/put

See merge request !18

Before this commit it was unclear whether a post request was successful
or not. We now return the content and status to make this clearer

Setra supports vouchertype being optional, so the client should as well.

Refactor models and methods for batches

See merge request !17

This allows more readable code, as we can do

  if batch.batch_progress == BatchProgressEnum.VALIDATION_FAILED:

instead of

  if batch.batch_progress == str(BatchProgressEnum.VALIDATION_FAILED):

or

  if batch.batch_progress == BatchProgressEnum.VALIDATION_FAILED.value:

Validation rule 12 defines period as an optional field. If empty, it
will be populated by master data.