Bug 20386

Summary: Improve warning and error messages for Search Engine Configuration
Product: Koha Reporter: Claire Gravely <claire_gravely>
Component: Searching - ElasticsearchAssignee: Claire Gravely <claire_gravely>
Status: CLOSED FIXED QA Contact:
Severity: enhancement    
Priority: P5 - low CC: fridolin.somers, nick
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 20386: Improve warning and error messages for Search Engine Configuration
Bug 20386: Improve warning and error messages for Search Engine Configuration
Bug 20386: Improve warning and error messages for Search Engine Configuration

Description Claire Gravely 2018-03-13 11:16:44 UTC
Warning and error messages on the search engine configuration page could be improved. /cgi-bin/koha/admin/searchenging/elasticsearch/mappings.pl
Comment 1 Claire Gravely 2018-03-13 11:20:24 UTC
Created attachment 72768 [details] [review]
Bug 20386: Improve warning and error messages for Search Engine Configuration

This patch updates a warning message and error message in the Search
engine configuration for Elasticsearch.

Note: elasticsearch does not need to be active in order to test.

To test warning message:
 1. go to: cgi-bin/koha/admin/searchengine/elasticsearch/mappings.pl
 2. note the warning message
 3. Apply patch
 4. Note the change in warning message. Confirm it makes sense.

To test error message:
  Apply patch and check the changed message in the template file mappings.tt and confirm it makes sense.
Comment 2 Nicolas Legrand 2018-03-14 09:34:05 UTC
Created attachment 72833 [details] [review]
Bug 20386: Improve warning and error messages for Search Engine Configuration

This patch updates a warning message and error message in the Search
engine configuration for Elasticsearch.

Note: elasticsearch does not need to be active in order to test.

To test warning message:
 1. go to: cgi-bin/koha/admin/searchengine/elasticsearch/mappings.pl
 2. note the warning message
 3. Apply patch
 4. Note the change in warning message. Confirm it makes sense.

To test error message:
  Apply patch and check the changed message in the template file mappings.tt and confirm it makes sense.

Signed-off-by: Nicolas Legrand <nicolas.legrand@bulac.fr>
Comment 3 Katrin Fischer 2018-03-18 21:58:24 UTC
Created attachment 73079 [details] [review]
Bug 20386: Improve warning and error messages for Search Engine Configuration

This patch updates a warning message and error message in the Search
engine configuration for Elasticsearch.

Note: elasticsearch does not need to be active in order to test.

To test warning message:
 1. go to: cgi-bin/koha/admin/searchengine/elasticsearch/mappings.pl
 2. note the warning message
 3. Apply patch
 4. Note the change in warning message. Confirm it makes sense.

To test error message:
  Apply patch and check the changed message in the template file mappings.tt and confirm it makes sense.

Signed-off-by: Nicolas Legrand <nicolas.legrand@bulac.fr>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 4 Jonathan Druart 2018-03-27 20:55:52 UTC
Pushed to master for 18.05, thanks to everybody involved!
Comment 5 Nick Clemens 2018-05-14 21:25:47 UTC
Pushed to stable for 17.11.06

Awesome work all!
Comment 6 Fridolin Somers 2018-05-22 09:21:38 UTC
Not pushed to 17.05.x