Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

QA-15202 - Augmented-search-ui broken when performing a search #90

Closed
jahia-ci opened this issue Jun 18, 2024 · 6 comments
Closed

QA-15202 - Augmented-search-ui broken when performing a search #90

jahia-ci opened this issue Jun 18, 2024 · 6 comments
Assignees
@hduchesne
Labels
affectVersion:augmented-search-3.6.0 affectVersion:augmented-search-3.7.0-SNAPSHOT Area:Augmented Search fixVersion:augmented-search-ui-3.1.0-SNAPSHOT Jira:QA msg-wip Priority:Major Scope-change SP:1 Sprint:TTY - Sprint 24.27 team-tty

Comments

@jahia-ci
Copy link
Contributor

jahia-ci commented Jun 18, 2024 •
edited
Loading

Imported from Jira, on Thu Jan 30, 2025
Issue: QA-15202 in project: QA DX
Priority: Major Type: Bug
Reporter: @ksagnard (Karine Sagnard)
Assignee: @hduchesne (Hervé Duchesne)
Created: Tue Jun 18, 2024, closed: Fri Jul 5, 2024
Status: Closed (resolution: Done)
Sprint: TTY - Sprint 24.27 started on Wed Jul 3, 2024 completed on Wed Jul 17, 2024

Links:

Attachments:

Steps to reproduce

clone git repository https://github.com/Jahia/preview-env

startup rgauthier local environment

cd preview-env/terraform/modules/ec2-docker/environments/rgauthier

bash local-startup.sh

wait for the startup to be completed

Access digitall home page

In the augmented-search field type "abc"

Current Behavior

The page is refreshed

No results are shown, the search field is now hidden

A error is displayed in browser console

SearchView.jsx:63 Uncaught TypeError: Cannot read properties of undefined (reading 'lang')
    at buildAndFireSearchEvent (SearchView.jsx:63:46)
    at eval (SearchView.jsx:78:13)
    at commitHookEffectListMount (react-dom.development.js:19731:26)
    at commitPassiveHookEffects (react-dom.development.js:19769:11)
    at HTMLUnknownElement.callCallback (react-dom.development.js:188:14)
    at Object.invokeGuardedCallbackDev (react-dom.development.js:237:16)
    at invokeGuardedCallback (react-dom.development.js:292:31)
    at flushPassiveEffectsImpl (react-dom.development.js:22853:9)
    at unstable_runWithPriority (scheduler.development.js:653:12)
    at runWithPriority$1 (react-dom.development.js:11039:10)
buildAndFireSearchEvent @ SearchView.jsx:63
eval @ SearchView.jsx:78
commitHookEffectListMount @ react-dom.development.js:19731
commitPassiveHookEffects @ react-dom.development.js:19769
callCallback @ react-dom.development.js:188
invokeGuardedCallbackDev @ react-dom.development.js:237
invokeGuardedCallback @ react-dom.development.js:292
flushPassiveEffectsImpl @ react-dom.development.js:22853
unstable_runWithPriority @ scheduler.development.js:653
runWithPriority$1 @ react-dom.development.js:11039
flushPassiveEffects @ react-dom.development.js:22820
performSyncWorkOnRoot @ react-dom.development.js:21737
eval @ react-dom.development.js:11089
unstable_runWithPriority @ scheduler.development.js:653
runWithPriority$1 @ react-dom.development.js:11039
flushSyncCallbackQueueImpl @ react-dom.development.js:11084
flushSyncCallbackQueue @ react-dom.development.js:11072
flushPendingDiscreteUpdates @ react-dom.development.js:21847
flushDiscreteUpdates @ react-dom.development.js:21827
finishEventHandler @ react-dom.development.js:764
batchedEventUpdates @ react-dom.development.js:798
dispatchEventForLegacyPluginEventSystem @ react-dom.development.js:3568
attemptToDispatchEvent @ react-dom.development.js:4267
dispatchEvent @ react-dom.development.js:4189
unstable_runWithPriority @ scheduler.development.js:653
runWithPriority$1 @ react-dom.development.js:11039
discreteUpdates$1 @ react-dom.development.js:21887
discreteUpdates @ react-dom.development.js:806
dispatchDiscreteEvent @ react-dom.development.js:4168
Show 27 more frames
Show less
react-dom.development.js:19527 The above error occurred in one of your React components:
    in Unknown (created by WithSearch)
    in WithSearch (created by WithSearch)
    in WithSearch (created by App)
    in SearchProvider (created by App)
    in AppConsider adding an error boundary to your tree to customize error handling behavior.
Visit https://fb.me/react-error-boundaries to learn more about error boundaries.
logCapturedError @ react-dom.development.js:19527
logError @ react-dom.development.js:19564
update.callback @ react-dom.development.js:20708
callCallback @ react-dom.development.js:12490
commitUpdateQueue @ react-dom.development.js:12511
commitLifeCycles @ react-dom.development.js:19883
commitLayoutEffects @ react-dom.development.js:22803
callCallback @ react-dom.development.js:188
invokeGuardedCallbackDev @ react-dom.development.js:237
invokeGuardedCallback @ react-dom.development.js:292
commitRootImpl @ react-dom.development.js:22541
unstable_runWithPriority @ scheduler.development.js:653
runWithPriority$1 @ react-dom.development.js:11039
commitRoot @ react-dom.development.js:22381
finishSyncRender @ react-dom.development.js:21807
performSyncWorkOnRoot @ react-dom.development.js:21793
eval @ react-dom.development.js:11089
unstable_runWithPriority @ scheduler.development.js:653
runWithPriority$1 @ react-dom.development.js:11039
flushSyncCallbackQueueImpl @ react-dom.development.js:11084
flushSyncCallbackQueue @ react-dom.development.js:11072
flushPendingDiscreteUpdates @ react-dom.development.js:21847
flushDiscreteUpdates @ react-dom.development.js:21827
finishEventHandler @ react-dom.development.js:764
batchedEventUpdates @ react-dom.development.js:798
dispatchEventForLegacyPluginEventSystem @ react-dom.development.js:3568
attemptToDispatchEvent @ react-dom.development.js:4267
dispatchEvent @ react-dom.development.js:4189
unstable_runWithPriority @ scheduler.development.js:653
runWithPriority$1 @ react-dom.development.js:11039
discreteUpdates$1 @ react-dom.development.js:21887
discreteUpdates @ react-dom.development.js:806
dispatchDiscreteEvent @ react-dom.development.js:4168
Show 33 more frames
Show less
react-dom.development.js:11102 Uncaught TypeError: Cannot read properties of undefined (reading 'lang')
    at buildAndFireSearchEvent (SearchView.jsx:63:46)
    at eval (SearchView.jsx:78:13)
    at commitHookEffectListMount (react-dom.development.js:19731:26)
    at commitPassiveHookEffects (react-dom.development.js:19769:11)
    at HTMLUnknownElement.callCallback (react-dom.development.js:188:14)
    at Object.invokeGuardedCallbackDev (react-dom.development.js:237:16)
    at invokeGuardedCallback (react-dom.development.js:292:31)
    at flushPassiveEffectsImpl (react-dom.development.js:22853:9)
    at unstable_runWithPriority (scheduler.development.js:653:12)
    at runWithPriority$1 (react-dom.development.js:11039:10)

We have the issue with augmented-search-3.6.0/augmented-search-ui 3.0.5 and augmented-search-3.7.0-SNAPSHOT/augmented-search-ui-3.1.0-SNAPSHOT

It happens on https://rgauthier.preview.sandbox.jahia.com/fr/home.html and on a new cloud instance, I do not have it on a local (non docker) installation

Desired Behavior

The search should return consistent results without error in browser logs

 

QA notes

reteste partially DMF-6183 : make sure that searches from simple AS form and advanced AS form are stored and displayed in kibana search dashboard
@jahia-ci
Copy link
Contributor Author

Comment by: @jahia-ci (Jahia CI) on Thu Jun 20, 2024 edited

We are now working on this issue and will provide an update soon.

@jahia-ci
Copy link
Contributor Author

Comment by: @hduchesne (Hervé Duchesne) on Tue Jun 25, 2024 edited

fix and merge done on the master branch -> augmented-search-ui-3.1.0-SNAPSHOT

@jahia-ci
Copy link
Contributor Author

Comment by: @Fgerthoffert (Francois Gerthoffert) on Wed Jun 26, 2024 edited

Test without and with jexperience

@jahia-ci
Copy link
Contributor Author

Comment by: @ksagnard (Karine Sagnard) on Wed Jun 26, 2024 edited

augmented-search-3.7.0-SN 
augmented-search-ui-3.1.0-SN
Heading 1 with jexperience started without jexperience
https://rgauthier.preview.sandbox.jahia.com/fr/home.html ok augmented search works fine
search terms are reported in kibana search dashboard  
cloud environement    

On the cloud environment we have the following error in the UI when trying to use augmented search in live :

image-2024-06-26-23-09-02-020.png

datadog logs :

java.util.concurrent.CompletionException: org.jahia.modules.graphql.provider.dxm.DataFetchingException: <ksa-test-82-jahiadevs*_jahia_as_v3__en__read> ElasticsearchStatusException<Elasticsearch exception [type=index_not_found_exception, reason=no such index [ksa-test-82-jahiadevs__jahia_as_v3__en_*read>]]

The issue is similar to the one discussed here: https://jahia.slack.com/archives/CDTMU869X/p1719243859603089?thread_ts=1719232801.273499&cid=CDTMU869X
@rknj (Michael De Checchi) do you know what are the next step for this?

@jahia-ci
Copy link
Contributor Author

jahia-ci commented Jul 2, 2024

Comment by: @kevinvandel (Kevin Vandel) on Tue Jul 2, 2024 edited

Indeed the issue you mention is the same as shortly explained here in BACKLOG-22935.

@jahia-ci
Copy link
Contributor Author

jahia-ci commented Jul 5, 2024

Comment by: @ksagnard (Karine Sagnard) on Fri Jul 5, 2024 edited

 

Retested on the cloud 

Heading 1 with jexperience started without jexperience
https://rgauthier.preview.sandbox.jahia.com/fr/home.html ok augmented search works fine
search terms are reported in kibana search dashboard –
cloud environement – ok augmented search works fine (we need to start manually an indexation before testing)

 

@jahia-ci jahia-ci closed this as completed Jul 5, 2024
@rknj rknj added the transfer:augmented-search-ui Label used to transfer issues between repositories label Feb 6, 2025
@jahia-ci jahia-ci removed the transfer:augmented-search-ui Label used to transfer issues between repositories label Feb 6, 2025
@jahia-ci jahia-ci transferred this issue from another repository Feb 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hduchesne hduchesne

Labels
affectVersion:augmented-search-3.6.0 affectVersion:augmented-search-3.7.0-SNAPSHOT Area:Augmented Search fixVersion:augmented-search-ui-3.1.0-SNAPSHOT Jira:QA msg-wip Priority:Major Scope-change SP:1 Sprint:TTY - Sprint 24.27 team-tty
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jahia-ci @rknj @hduchesne