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.

Sign up for GitHub

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

Jump to bottom

hotfix for select(index) #369

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

hotfix for select(index) #369

wants to merge 2 commits into from

Conversation

maksimbykov
Copy link

@maksimbykov maksimbykov commented Aug 31, 2022 •
edited
Loading

Fix #356

@maksimbykov
Copy link
Author

@TarekRaafat please review this when you'll have time, thanks❤️

@maksimbykov maksimbykov changed the title hotfix hotfix for select(index) Aug 31, 2022
@maksimbykov maksimbykov mentioned this pull request Sep 1, 2022
Open
@AndrewElans
Copy link

Any news on this bug?

@AndrewElans
Copy link

AndrewElans commented Apr 28, 2023 •
edited
Loading

I guess the reason why this TypeError: Cannot set properties of undefined (setting 'event') is fired is because select(index) is called when function init (ctx) is not yet complete.

I have fixed in my case by placing select(index) in the results event which seems to run after init is finished.

Before I tried to call select(index) on window.load and other events but in vain. Also in init event the same error was generated. I guess that even if the init event is fired, the init promise may still be not returned.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

API Call to select an resultItem is not working
2 participants
@maksimbykov @AndrewElans