Skip to content

Update concepts.md #780

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

Merged
merged 1 commit into from
Mar 26, 2023
Merged

Update concepts.md #780

merged 1 commit into from
Mar 26, 2023

Conversation

j-medland
Copy link
Contributor

There appeared to be a small bug in the example code where the feature.showItemByIndex() never called the proxy'd function that it created so the function exported from the module didn't actually do anything.

feature.showItemByIndex() function calls proxy'd function that is created in final example
@jsf-clabot
Copy link

jsf-clabot commented Oct 30, 2018

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

Base automatically changed from master to main March 8, 2021 17:15
@Krinkle Krinkle merged commit 6e6e66b into jquery:main Mar 26, 2023
Krinkle pushed a commit that referenced this pull request Apr 9, 2024
`feature.showItemByIndex()` function calls proxy'd function that is created in final example.

Closes #780.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants