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

fix(insights): export Insights helper in the ESM build #4261

Merged
merged 1 commit into from Dec 16, 2019

Conversation

francoischalifour
Copy link
Member

@francoischalifour francoischalifour commented Dec 16, 2019

We never exported the insights helper in the ESM build, so the function was never usable. This fixes it by exporting the insights helper as we do for highlight and snippet.

@francoischalifour francoischalifour requested a review from a team December 16, 2019 10:29
@ghost ghost requested review from tkrugg and yannickcr and removed request for a team December 16, 2019 10:30
@francoischalifour francoischalifour merged commit 20649af into develop Dec 16, 2019
@francoischalifour francoischalifour deleted the fix/insights-export-esm-build branch December 16, 2019 12:36
Haroenv added a commit that referenced this pull request Dec 20, 2019
### Bug Fixes

* **configureRelatedItems:** use `facetFilters` to exclude `obje… ([#4264](#4264)) ([9bc66cf](9bc66cf))
* **index:** fix warning for widgets sharing connectors ([#4260](#4260)) ([ec97b4a](ec97b4a))
* **insights:** export Insights helper in the ESM build ([#4261](#4261)) ([20649af](20649af))
* **insights:** move 'insightsClient not provided error' to wrapper level ([#4254](#4254)) ([15d38dd](15d38dd))

### Features

* **insights:** add hogan helper ([#4253](#4253)) ([85739d7](85739d7))
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.

None yet

3 participants