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.

Instead of mixing the Batch model for batches sent *to* with batches
received *from* SETRA, we switch to using an InputBatch and an
OutputBatch. The InputBatch contains the fields that can be used when
sending a Batch (with Vouchers and Transactions), and the OutputBatch
contains all the information one receives from SETRA when fetching a
single batch from the batch/ endpoint, or multiple using the search
functionality.

The get_batch method is reduced to only fetch a single batch, while the
new search_batches method takes the role of searching based on creation
date and interface. In addition, the return_list_of_obj parameter is
removed in favor of the return_objects class attribute.

This commit expects that SETRA includes the id field in Batches.

Bugfix/optional fields in models.Parameter

See merge request !16

Also a little tidying

FSHO-19: Added update batch method. And fixed format to match the post_new_batch method

See merge request !14