searchdiscovery / client-jnj-askgs-ga4-dl-spec

An Apollo generated Event Driven Data Layer
1 stars 0 forks source link

Onsite Search Succeeded #14

Open ajramosSDI opened 1 year ago

ajramosSDI commented 1 year ago

This event should fire in the event of a successful search. Note: this event should fire in a sequential order AFTER Onsite Search Performed.md

Onsite Search Succeeded.md

mgarzonjnj commented 1 year ago

Image

davisbradleyj commented 1 year ago

@mgarzonjnj

Is there a specific link in the dev site that our team should use for validation?

mandyjohnston03 commented 11 months ago

Event is firing as expected. The following parameters need attention:

'results_count' is returning 20 in every scenario and seems to be hardcoded. This should be dynamic.

mgarzonjnj commented 11 months ago

Result_Count - Because of how our search works, in that it indexes 20 results always and only displays another 20 (maxes out at 100) if the user scrolls down, we're unable to make this more dynamic. However, when results are less than 20, I have asked the team to provide that value here.

mgarzonjnj commented 11 months ago

Result count for searches with less than 20 results, the count should be updated. Can you please retest @ACasallaJnJ @mandyjohnston03

davisbradleyj commented 11 months ago

Confirming a results_count is now better reflecting the returned items from a search, pushing this forward to GTM Validation

Image

mandyjohnston03 commented 11 months ago

Passed in pre-dev

Image

davisbradleyj commented 11 months ago

https://jnjgsportal--uat2.sandbox.my.site.com/s/?language=es

mandyjohnston03 commented 11 months ago

Passed in UAT

Image