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

fix: coalesce action methods to reduce noise in autocapture #198

Conversation

PouriaAmini
Copy link
Contributor

@PouriaAmini PouriaAmini commented Jul 19, 2024

Summary

This PR coalesces action methods fired by a target element when a user interacts with that element.

Checklist

  • Does your PR title have the correct title format?
  • Does your PR have a breaking change?: no

@PouriaAmini PouriaAmini requested a review from crleona July 19, 2024 15:28
@PouriaAmini PouriaAmini self-assigned this Jul 19, 2024
@PouriaAmini
Copy link
Contributor Author

Hi @crleona, this PR is ready for your review when you get a chance!
After syncing with Alan, I confirmed that these events are indeed what we want to capture when a user interacts with the app.

@PouriaAmini PouriaAmini changed the title fix: track button clicks only on touchupinside fix: coalesce action methods to reduce noise in autocapture Jul 22, 2024
@PouriaAmini PouriaAmini removed their assignment Jul 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant