Skip to content
This repository has been archived by the owner on Dec 16, 2022. It is now read-only.

feat(js): add typescript to the js template #464

Merged
merged 3 commits into from
Aug 24, 2021
Merged

Conversation

Haroenv
Copy link
Contributor

@Haroenv Haroenv commented Aug 23, 2021

Since algolia/instantsearch#4844 the UMD has type definitions exposed, which we can use in codesandbox, even when the file itself isn't written in TypeScript, or using modules

@Haroenv Haroenv requested review from a team, eunjae-lee and francoischalifour and removed request for a team August 23, 2021 12:00
eunjae-lee
eunjae-lee previously approved these changes Aug 23, 2021
@eunjae-lee eunjae-lee dismissed their stale review August 23, 2021 12:36

waiting for #4844 to be applied here

Copy link
Contributor Author

@Haroenv Haroenv left a comment

Choose a reason for hiding this comment

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

with algolia/instantsearch#4844 the code can be simplified

Copy link
Contributor

@eunjae-lee eunjae-lee left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@Haroenv Haroenv merged commit 9b16b13 into master Aug 24, 2021
@Haroenv Haroenv deleted the feat/template-ts branch August 24, 2021 12:21
aymeric-giraudet pushed a commit to algolia/instantsearch that referenced this pull request Dec 14, 2022
…arch-app#464)

* feat(js): add typescript to the js template

based on the (for now) not exported as TS https://github.com/algolia/instantsearch.js/blob/master/src/index.ts

* update import method

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

Successfully merging this pull request may close these issues.

3 participants