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

Update index.md #35865

Closed
wants to merge 1 commit into from
Closed

Update index.md #35865

wants to merge 1 commit into from

Conversation

deekeh
Copy link
Contributor

@deekeh deekeh commented Sep 13, 2024

Changed confusing code

Description

The existing code would not run if properly if copy-pasted as it is. The developer should be shown the relation between adding the listener and the handler.

Motivation

Additional details

Related issues and pull requests

Changed confusing code
@deekeh deekeh requested a review from a team as a code owner September 13, 2024 07:16
@deekeh deekeh requested review from Elchi3 and removed request for a team September 13, 2024 07:16
@github-actions github-actions bot added Content:WebAPI Web API docs size/xs [PR only] 0-5 LoC changed labels Sep 13, 2024
Copy link
Contributor

Copy link
Member

@Josh-Cena Josh-Cena left a comment

Choose a reason for hiding this comment

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

Thanks, but this is the standard event listener syntax format. You either do the first line or the second line; they have no relationship between each other.

@Josh-Cena Josh-Cena closed this Sep 13, 2024
@deekeh deekeh deleted the patch-1 branch September 13, 2024 14:37
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:WebAPI Web API docs size/xs [PR only] 0-5 LoC changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants