Bug 28400 - Add response_message column to message_queue table
Summary: Add response_message column to message_queue table
Status: Signed Off
Alias: None
Product: Koha
Classification: Unclassified
Component: Notices (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement
Assignee: Johanna Räisä
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-05-20 11:37 UTC by Johanna Räisä
Modified: 2025-01-25 21:33 UTC (History)
1 user (show)

See Also:
Change sponsored?: Sponsored
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
This enhancement adds a 'response_message' field to the 'message_queue' table. This can be used to save the message response. For example, some SMS providers give an ID in the response body that could be used to track the message status - this change adds a field where this information can be saved.
Version(s) released in:
Circulation function:


Attachments
Bug 24800: Add response_message column to message_queue table (1.49 KB, patch)
2024-07-29 09:17 UTC, Johanna Räisä
Details | Diff | Splinter Review
Bug 28400: Add response_message column to message_queue table (1.49 KB, patch)
2024-07-29 09:20 UTC, Johanna Räisä
Details | Diff | Splinter Review
Bug 28400: Add response_message column to message_queue table (2.29 KB, patch)
2024-07-29 11:18 UTC, Johanna Räisä
Details | Diff | Splinter Review
Bug 28400: Add response_message column to message_queue table (2.33 KB, patch)
2025-01-25 21:26 UTC, David Nind
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Johanna Räisä 2021-05-20 11:37:33 UTC
Some SMS providers give an id in response body which you can use to track message status. Now message_queue table doesn't have any field where this could be put.
Comment 1 Johanna Räisä 2021-08-09 07:22:51 UTC
(In reply to Johanna Räisä from comment #0)
> Some SMS providers give an id in response body which you can use to track
> message status. Now message_queue table doesn't have any field where this
> could be put.

Probably saving the whole response body is the best thing to do since there can be more useful data than just id.
Comment 2 Johanna Räisä 2024-07-29 05:36:44 UTC
Updated the title, since this can be used for all the interfaces to save a response.
Comment 3 Johanna Räisä 2024-07-29 09:17:08 UTC
Created attachment 169812 [details] [review]
Bug 24800: Add response_message column to message_queue table

Test Plan:
1) Run the updatedatabase.pl script
2) Run update_dbix_class_files.pl
3) Check the message_queue table for the new column

Sponsored-by: Koha-Suomi Oy
Comment 4 Johanna Räisä 2024-07-29 09:20:27 UTC
Created attachment 169813 [details] [review]
Bug 28400: Add response_message column to message_queue table

Test Plan:
1) Run the updatedatabase.pl script
2) Run update_dbix_class_files.pl
3) Check the message_queue table for the new column

Sponsored-by: Koha-Suomi Oy
Comment 5 Johanna Räisä 2024-07-29 11:18:03 UTC
Created attachment 169821 [details] [review]
Bug 28400: Add response_message column to message_queue table

Test Plan:
1) Run the updatedatabase.pl script
2) Run update_dbix_class_files.pl
3) Check the message_queue table for the new column

Sponsored-by: Koha-Suomi Oy
Comment 6 David Nind 2025-01-25 21:26:08 UTC
Created attachment 177147 [details] [review]
Bug 28400: Add response_message column to message_queue table

Test Plan:
1) Run the updatedatabase.pl script
2) Run update_dbix_class_files.pl
3) Check the message_queue table for the new column

Sponsored-by: Koha-Suomi Oy
Signed-off-by: David Nind <david@davidnind.com>
Comment 7 David Nind 2025-01-25 21:33:56 UTC
Testing notes (using KTD):
1. SQl to show the columns for a table: DESCRIBE message_queue;