Add notification triggers support#57
Conversation
GormFrank
left a comment
There was a problem hiding this comment.
This is missing the focusToView update so that the trigger receives focus upon display instead of being bypassed to the Query summary
|
Closes #36 |
BeraJosh
left a comment
There was a problem hiding this comment.
Notice changes, no security or risk is observed.
Requested Changes will be locally tested before merge
Approved
|
@bblaisATcoveo Can you please add steps to reproduce? If I need to change stuff on the page to get a working example locally, this will need to be addressed by creating a page with a working example. Thanks! |
|
@Garneauma Test cases were added. Notifications are entirely driven by configs on the backend only so we can use srb-en.html or sb-fr.html for testing |
|
@bblaisATcoveo What are the patterns available so I can test? |
|
@bblaisATcoveo As-tu un exemple pour en français? J'ai juste "sign in", mais le message est en anglais dans la page en français. |
|
@GormFrank Added a notification trigger for French "se connecter". It works as expected. |
Add the possibility to display an information message on top of the search box to provide additional details using the notification trigger mechanism.
TEST CASE 1
Prerequisite : a 'notification trigger' must be configured on the backend with proper conditions
1- Open the search results page, english version
2- Perform a search that match the condition of a trigger
3- A notification should appear under the search box in english
TEST CASE 2
Prerequisite : a 'notification trigger' must be configured on the backend with proper conditions
1- Open the search results page, french version
2- Perform a search that match the condition of a trigger
3- A notification should appear under the search box in french