Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support for flere dokumenter i bestillinger og maler #3733

Open
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

krharum
Copy link
Contributor

@krharum krharum commented Jan 28, 2025

This pull request includes changes to the DokarkivClient and HistarkClient classes to improve error handling, update progress tracking, and refactor the handling of multiple responses. Additionally, there are changes to the HistarkConsumer and HistarkPostCommand classes to modify the request handling.

Improvements to DokarkivClient:

  • Added updateProgress method to update the progress status for Dokarkiv operations.
  • Refactored getStatus and saveTransaksjonId methods to handle lists of DokarkivResponse objects instead of single responses. [1] [2]

Improvements to HistarkClient:

  • Added updateProgress method to update the progress status for Histark operations.
  • Refactored getStatus and saveTransaksjonId methods to handle lists of HistarkResponse objects instead of single responses.

Changes to HistarkConsumer and HistarkPostCommand:

  • Changed postHistark method in HistarkConsumer to return a Mono instead of a Flux.
  • Updated HistarkPostCommand to use fromMultipartData for request body creation and to return a Mono instead of a Flux. [1] [2]

These changes enhance the robustness and efficiency of the Dokarkiv and Histark clients by improving error handling, progress tracking, and response management.This pull request includes significant updates to the DokarkivClient and HistarkClient classes, along with various other changes to improve functionality and code consistency. The most important changes include the introduction of new methods for updating progress, handling multiple responses, and renaming classes for better clarity.

Improvements to DokarkivClient and HistarkClient:

  • DokarkivClient.java:

    • Added updateProgress method to handle progress updates for multiple environments.
    • Modified gjenopprett method to handle lists of Dokarkiv requests and responses, improving error handling and status updates.
    • Updated getStatus and saveTransaksjonId methods to work with lists of DokarkivResponse objects. [1] [2]
  • HistarkClient.java:

    • Added updateProgress method for progress updates.
    • Modified gjenopprett method to handle lists of Histark requests and responses, improving error handling and status updates.
    • Updated getStatus and saveTransaksjonId methods to work with lists of HistarkResponse objects. [1] [2]

Codebase consistency and clarity:

  • Renamed JoarkHistarkTransaksjon to HistarkTransaksjon for better clarity.
  • Updated BestilteKriterier and RsDollyBestilling classes to use lists for dokarkiv and histark fields. [1] [2]
  • Removed unnecessary field mappings in DollyRequest2MalBestillingMappingStrategy.

Additional changes:

  • Added MigrationConfig class to handle document migration on application startup.
  • Minor formatting adjustments in Bestilling.java for better readability.…eploy-test-dolly-backend
  • Implemented methods to migrate bestillinger, malbestillinger, and transaksjon mappings.
  • Added JSON processing for converting bestillinger and transaksjon mappings.
  • Updated Bestilling entity with

…eploy-test-dolly-backend

- Implemented methods to migrate bestillinger, malbestillinger, and transaksjon mappings.
- Added JSON processing for converting bestillinger and transaksjon mappings.
- Updated `Bestilling` entity with
@krharum krharum requested a review from a team as a code owner January 28, 2025 07:48
@krharum krharum changed the title feat: Add migration logic for dokumenter in MigrateDokumentService #d… Support for flere dokumenter i bestillinger og maler Jan 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant