Right now, there's not link back to display the edifact invoice message associated with a Koha invoice.. it would be helpful to have a quick link back from the Invoice display page.
Created attachment 183902 [details] [review] Bug 40333: Add EDIFACT message link to invoice display page This enhancement adds a link from the invoice display page back to the EDIFACT message that was used to create the invoice, improving workflow and traceability for acquisitions staff. Changes include: - Display EDIFACT message information on invoice page when available - Show message type, transfer date, status, and filename - Add "View EDIFACT message" button that opens modal with raw message - Display EDIFACT processing errors directly on invoice page - Create reusable modal components for EDIFACT message display The implementation checks if the EDIFACT system is enabled and if the invoice has an associated message_id before displaying the EDIFACT section. Processing errors are highlighted in a warning box with clear formatting. To test: 1. Enable EDIFACT system preference 2. Create an invoice linked to an EDIFACT message 3. View the invoice - EDIFACT section should appear after vendor info 4. Click "View EDIFACT message" to see the raw EDIFACT data in a modal 5. If there are processing errors, they should be displayed clearly
Created attachment 183903 [details] Example display
Oh, I like this idea...
Created attachment 183941 [details] [review] Bug 40333: Add EDIFACT message view to invoice display page This enhancement adds an EDIFACT message section to the invoice display page. If there's a linked EDI Message for this invice, we display the basic details along with any parsing errors that may have been recorded and we add a button to trigger the display of the raw message in a modal. Changes include: - Display EDIFACT message information on invoice page when available - Show message type, transfer date, status, and filename - Add "View EDIFACT message" button that opens modal with raw message - Display EDIFACT processing errors directly on invoice page - Create reusable modal components for EDIFACT message display The implementation checks if the EDIFACT system is enabled and if the invoice has an associated message_id before displaying the EDIFACT section. Processing errors are highlighted in a warning box with clear formatting.
Created attachment 183942 [details] [review] Bug 40333: Tidy commit The commit hook was deliverately disabled for the previous commit and this commit added distinctly to ensure clarity between the functional code change and the massive tidy that the hook created. In that last commit we remove the last remaining TT variable from the <script> block in that template and as such we unlock the JS tidying on that block.. nice, except it hides the functional change in the mass commit of whitespace, quoting and newline changes.
Created attachment 184385 [details] [review] Bug 40333: Add EDIFACT message view to invoice display page This enhancement adds an EDIFACT message section to the invoice display page. If there's a linked EDI Message for this invice, we display the basic details along with any parsing errors that may have been recorded and we add a button to trigger the display of the raw message in a modal. Changes include: - Display EDIFACT message information on invoice page when available - Show message type, transfer date, status, and filename - Add "View EDIFACT message" button that opens modal with raw message - Display EDIFACT processing errors directly on invoice page - Create reusable modal components for EDIFACT message display The implementation checks if the EDIFACT system is enabled and if the invoice has an associated message_id before displaying the EDIFACT section. Processing errors are highlighted in a warning box with clear formatting. Sponsored-by: Open Fifth <https://openfith.co.uk> Signed-off-by: Hannah Dunne-Howrie <hdunne-howrie@westminster.gov.uk>
Created attachment 184386 [details] [review] Bug 40333: Tidy commit The commit hook was deliverately disabled for the previous commit and this commit added distinctly to ensure clarity between the functional code change and the massive tidy that the hook created. In that last commit we remove the last remaining TT variable from the <script> block in that template and as such we unlock the JS tidying on that block.. nice, except it hides the functional change in the mass commit of whitespace, quoting and newline changes. Sponsored-by: Open Fifth <https://openfith.co.uk> Signed-off-by: Hannah Dunne-Howrie <hdunne-howrie@westminster.gov.uk>
Created attachment 184387 [details] [review] Bug 40333: (follow-up) Terminology The envelope level of EDIFACT is called an 'Interchange'. This patch updates our UI terminology to match the official protocol terminology as what we're actually displaying here is the Interchange file that contains the related message for the Invoice. The interchange file will often contain other messages of the same type that came through at the same time of the this invoice generation. Sponsored-by: Open Fifth <https://openfith.co.uk> Signed-off-by: Hannah Dunne-Howrie <hdunne-howrie@westminster.gov.uk>
This is a fantastic feature and saves our library systems team time when supporting queries as we don't need to spend extra time combing the edi messages area for the corresponding file to check for the edifact included regarding a particular invoice. It is working as expected and so far and we are grateful for it's inclusion in our instance of Koha.