valtimo-backend-libraries icon indicating copy to clipboard operation
valtimo-backend-libraries copied to clipboard

feat(documenten): added operations to bulk upload documents, check th…

Open sdegroot opened this issue 1 year ago • 0 comments

Added operations to bulk upload documents, check the bulk upload status and get a report on the upload

Describe the changes

There is no GitHub issue. I was missing a few endpoints from open-zaak to import bulk documents; see https://open-zaak.readthedocs.io/en/stable/api/experimental.html#id2

I added a few things to make testing of these libs easier.

  • TestOutboxService - which allows to check the outbox messages without relying on mocks; you may want to promote this for more use (or not)
  • Added a few equals methods that were missing. In general I noticed that this is missing often and that makes testing quite a bit harder.

I have not been able to test it against a running open-zaak as I do not know how I can build Valtimo locally. Maybe someone can help me with that?

Breaking changes

  • [x] The contribution only contains changes that are not breaking.

Documentation

  • [ ] Release notes have been written for these changes.

Link to the pull request in the Valtimo documentation repository:

New features or changes that have been introduced have been documented.

  • [ ] Yes
  • [ ] Not applicable
  • [x] Not documented yet; maybe something for a technical writer?

Tests

Unit tests have been added that cover these changes

  • [x] Yes
  • [ ] Not applicable

Integration tests have been added that cover these changes

  • [ ] Yes
  • [x] Not applicable

Security

The Secure by Default principle has been applied to these changes

  • [x] Yes
  • [ ] Not applicable

Added or changed REST API endpoints have authentication and authorization in place

  • [ ] Yes
  • [x] Not applicable

Valtimo access control checks have been implemented

  • [x] Yes
  • [ ] Not applicable

Dependencies

Newly added dependencies do not introduce known vulnerabilities/CVE's and are in line with the Valtimo license

  • [ ] Yes
  • [x] Not applicable

sdegroot avatar Apr 08 '25 14:04 sdegroot