Skip to content

Conversation

@shubham1g5
Copy link
Contributor

Summary

  • Adds form schema to Form analytics Event
  • Adds a new event for form entry

PR Checklist

  • If I think the PR is high risk, "High Risk" label is set
  • I have confidence that this PR will not introduce a regression for the reasons below
  • Do we need to enhance manual QA test coverage ? If yes, "QA Note" label is set correctly
  • Does the PR introduce any major changes worth communicating ? If yes, "Release Note" label is set and a "Release Note" is specified in PR description.

Automated test coverage

NA

Safety story

PR mostly Adds a fails safe method and report to Analytics.

@shubham1g5 shubham1g5 merged commit a573f7a into dv/connect_initial Feb 6, 2025
1 of 2 checks passed
@shubham1g5 shubham1g5 deleted the form_analytics branch February 6, 2025 17:14
@shubham1g5 shubham1g5 mentioned this pull request Feb 7, 2025
4 tasks
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.

3 participants