FOLIO-FSE / FOLIO-FSE/folio_migration_tools

Fix Order Notes Mapper

Open
#642 1 comment 0 reactions 0 assignees View on GitHub
bug Orders
Dominant language
Python
Stars
14
Forks
10
Avg merge
55m
Merged PRs (30d)
7

Description

Order notes are not currently being mapped properly in the tools. Need to:

- Add `self.notes_mapper.map_notes()` to `OrderMapper.perform_addtional_mapping()`
- Notes should be linked to the `orderLine`, not the `order`
- How do we handle mapping for multiple order lines? We don't currently set deterministic UUIDs for orderLines
- Add `self.extradatawriter.flush()` to `OrdersTransformer.wrap_up()`

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.