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

feat: add submit btn click event for default register page #1176

Merged
merged 1 commit into from
Feb 27, 2024

Conversation

zainab-amir
Copy link
Contributor

@zainab-amir zainab-amir commented Feb 27, 2024

Description

To compare the control and variation group for the A/B test where we are measuring the impact of removing username from registration form, added event to default "Register for free" button clicks.

JIRA

https://2u-internal.atlassian.net/browse/VAN-1827

How Has This Been Tested?

Locally

@zainab-amir zainab-amir requested a review from a team as a code owner February 27, 2024 09:07
Copy link

codecov bot commented Feb 27, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.33%. Comparing base (e1f4898) to head (4140608).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1176      +/-   ##
==========================================
+ Coverage   84.23%   84.33%   +0.09%     
==========================================
  Files         127      127              
  Lines        2392     2394       +2     
  Branches      678      680       +2     
==========================================
+ Hits         2015     2019       +4     
+ Misses        357      355       -2     
  Partials       20       20              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@zainab-amir zainab-amir merged commit e4ac128 into master Feb 27, 2024
7 checks passed
@zainab-amir zainab-amir deleted the sajjad/VAN-1827-add-event branch February 27, 2024 09:24
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