Skip to content

Commit

Permalink
Revert "feat: add register intent prop for login segement call (opene…
Browse files Browse the repository at this point in the history
…dx#34891)" (openedx#35021)

This reverts commit a2aa6bd.
  • Loading branch information
syedsajjadkazmii authored Jun 24, 2024
1 parent 2adee01 commit 5e3181c
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 6 deletions.
4 changes: 1 addition & 3 deletions openedx/core/djangoapps/user_authn/views/login.py
Original file line number Diff line number Diff line change
Expand Up @@ -363,15 +363,13 @@ def _track_user_login(user, request):
'MailChimp': False
}
)
register_intent = request.POST.get('register_intent') == 'true'
segment.track(
user.id,
"edx.bi.user.account.authenticated",
{
'category': "conversion",
'label': request.POST.get('course_id'),
'provider': None,
'register_intent': register_intent,
'provider': None
},
)

Expand Down
4 changes: 1 addition & 3 deletions openedx/core/djangoapps/user_authn/views/tests/test_login.py
Original file line number Diff line number Diff line change
Expand Up @@ -1041,7 +1041,6 @@ class LoginSessionViewTest(ApiTestCase, OpenEdxEventsTestMixin):
USERNAME = "bob"
EMAIL = "[email protected]"
PASSWORD = "password"
REGISTER_INTENT = 'true'

@classmethod
def setUpClass(cls):
Expand Down Expand Up @@ -1118,7 +1117,6 @@ def test_login(self, include_analytics, mock_segment):
data = {
"email": self.EMAIL,
"password": self.PASSWORD,
"register_intent": self.REGISTER_INTENT,
}
if include_analytics:
track_label = "edX/DemoX/Fall"
Expand Down Expand Up @@ -1147,7 +1145,7 @@ def test_login(self, include_analytics, mock_segment):
mock_segment.track.assert_called_once_with(
expected_user_id,
'edx.bi.user.account.authenticated',
{'category': 'conversion', 'provider': None, 'label': track_label, 'register_intent': True}
{'category': 'conversion', 'provider': None, 'label': track_label}
)

def test_login_with_username(self):
Expand Down

0 comments on commit 5e3181c

Please sign in to comment.