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.

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

Fixed Shopify Chat, Shopify Hydrogen, Global-e and ROUTE Detection #20

Merged
merged 5 commits into from
Apr 1, 2024

Conversation

rockeynebhwani
Copy link

@rockeynebhwani rockeynebhwani commented Mar 25, 2024

Shopify Chat loading changed. Core Web Vitals technology report was showing a big drop
Also Updated Shopify Hydrogen logic. Example - https://www.honeylove.com/en-gr/
Update logic for Route app. Core Web Vitals technoloogy report had a big drop for Route as well
Updated Global-e detection as it was missing certain type of implementations. Example - https://beekman1802.com/

Shopify Chat loading changed. Core Web Vitals technology was showing a big drop
There was a big drop in origins for Route app in Core Web Vitals Technology report. Fixed that
@rockeynebhwani rockeynebhwani changed the title Fixed Shopify Chat Detection Fixed Shopify Chat and ROUTE Detection Mar 25, 2024
Example site where detection was not working -https://beekman1802.com/
@rockeynebhwani rockeynebhwani changed the title Fixed Shopify Chat and ROUTE Detection Fixed Shopify Chat, Global-e and ROUTE Detection Mar 27, 2024
Example Site where this was not working - https://www.honeylove.com/en-gr/
@rockeynebhwani rockeynebhwani changed the title Fixed Shopify Chat, Global-e and ROUTE Detection Fixed Shopify Chat, Shopify Hydrogen, Global-e and ROUTE Detection Mar 27, 2024
@rviscomi
Copy link
Member

rviscomi commented Apr 1, 2024

Thanks for tending to these things @rockeynebhwani!

@rviscomi rviscomi merged commit 4075752 into HTTPArchive:main Apr 1, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants