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

Analytics - Track submit event after validation #1904

Merged

Conversation

araratthehero
Copy link
Contributor

@araratthehero araratthehero commented Nov 27, 2024

Description

We were tracking the submit event regardless if the component data was valid or not, but this PR fixes it. Here we track events when there are values emitted in the submitFlow, which happens when the onSubmit() is called and the component is valid.

Checklist

  • PR is labelled
  • Code is unit tested
  • Changes are tested manually

COAND-1038

@araratthehero araratthehero added the Chore [PRs only] Indicates any task that does not need to be mentioned in the public release notes label Nov 27, 2024
@araratthehero araratthehero force-pushed the feature/analytics_track_submit_after_validation branch 3 times, most recently from 2ae379a to 3aeaf1c Compare November 27, 2024 15:58
@araratthehero araratthehero force-pushed the feature/analytics_track_submit_after_validation branch 3 times, most recently from 5920a31 to 80cf8b6 Compare December 11, 2024 14:18
@araratthehero araratthehero marked this pull request as ready for review December 11, 2024 14:18
@araratthehero araratthehero requested a review from a team as a code owner December 11, 2024 14:18
@araratthehero araratthehero force-pushed the feature/analytics_track_submit_after_validation branch 2 times, most recently from 3aff097 to f676b53 Compare December 11, 2024 15:19
@araratthehero araratthehero force-pushed the feature/analytics_track_submit_after_validation branch from f676b53 to 99b525f Compare December 12, 2024 10:04
@araratthehero araratthehero force-pushed the feature/analytics_track_submit_after_validation branch from 99b525f to a0074bb Compare December 12, 2024 11:35
Copy link

✅ No public API changes

Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
79.4% Coverage on New Code (required ≥ 80%)
5.0% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

@araratthehero araratthehero requested a review from jreij December 17, 2024 09:53
@araratthehero araratthehero merged commit d32070e into develop Dec 17, 2024
11 of 12 checks passed
@araratthehero araratthehero deleted the feature/analytics_track_submit_after_validation branch December 17, 2024 13:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Chore [PRs only] Indicates any task that does not need to be mentioned in the public release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants