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

docs: The text is confusing in a note. #4296

Open
mariocorzo opened this issue Dec 6, 2024 · 1 comment
Open

docs: The text is confusing in a note. #4296

mariocorzo opened this issue Dec 6, 2024 · 1 comment
Assignees
@felangel
Labels
documentation Documentation requested

Comments

@mariocorzo
Copy link

This note is confusing because in onChange the order of invocation is reversed compared to onError.

image
@mariocorzo mariocorzo added the documentation Documentation requested label Dec 6, 2024
@felangel
Copy link
Owner

Sorry for the confusion, the note is correct (it calls attention to the invocation not the output) but I agree it's confusing so will reword it 👍

@felangel felangel self-assigned this Dec 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@felangel felangel

Labels
documentation Documentation requested
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mariocorzo @felangel