Skip to content
This repository was archived by the owner on Oct 20, 2023. It is now read-only.

API changes #106

Merged
merged 4 commits into from
Oct 10, 2019
Merged

API changes #106

merged 4 commits into from
Oct 10, 2019

Conversation

antoinegrant
Copy link
Member

I brought the API changes that @rickitan made in #96

@antoinegrant
Copy link
Member Author

antoinegrant commented Oct 10, 2019

I need to update the README with the API changes.

Do we have new docs (current docs) that reflect these changes?

@antoinegrant
Copy link
Member Author

Ok the REAME has been updated. All good for a round of reviews.

Copy link

@BrendaSegalShopify BrendaSegalShopify left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks soooo much for your help Antoine! ❤️

@antoinegrant
Copy link
Member Author

Comments addressed! 👍

Copy link

@carolineschnapp carolineschnapp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I 🎩 -ed in Debut, and it works great.

Still the same bug in Debut, but likely related to Debut, not Theme Scripts:

Search is broken on product pages

Copy link

@BrendaSegalShopify BrendaSegalShopify left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks so much for doing this!

@antoinegrant antoinegrant merged commit 0cc9bb8 into master Oct 10, 2019
@antoinegrant antoinegrant deleted the theme-predictive-search/api-update branch October 10, 2019 21:13
Copy link
Contributor

@rickitan rickitan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants