-
Notifications
You must be signed in to change notification settings - Fork 143
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
Add events for recommendations #1039
Add events for recommendations #1039
Conversation
78abaef
to
04ae5e7
Compare
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## recommendations-v2 #1039 +/- ##
=====================================================
Coverage ? 85.98%
=====================================================
Files ? 121
Lines ? 2291
Branches ? 658
=====================================================
Hits ? 1970
Misses ? 310
Partials ? 11 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good.
import { DEFAULT_REDIRECT_URL } from '../data/constants'; | ||
|
||
const RecommendationsPage = ({ location }) => { | ||
const { formatMessage } = useIntl(); | ||
const registrationResponse = location.state?.registrationResult; | ||
const educationLevel = EDUCATION_LEVEL_MAPPING[location.state?.educationLevel]; | ||
const userId = location.state?.userId; | ||
const userId = location.state?.userId || 1234; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do we need to set the default value here for userId
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, this was added for debugging.
68774ca
to
fbea2ea
Compare
* feat: add events for recommendations (#1039) * fix: Post registration feedback changes fix Post registration recommendations background color and label VAN-1616 --------- Co-authored-by: Zainab Amir <zainab.amir@arbisoft.com>
* feat: add events for recommendations (#1039) * fix: Post registration feedback changes fix Post registration recommendations background color and label VAN-1616 --------- Co-authored-by: Zainab Amir <zainab.amir@arbisoft.com>
* feat: add events for recommendations (#1039) * fix: Post registration feedback changes fix Post registration recommendations background color and label VAN-1616 --------- Co-authored-by: Zainab Amir <zainab.amir@arbisoft.com>
* feat: add personalized recommendations (#1024) * use Algolia for personalized recommendations * show personalized recommendations to use that have consented to functional cookies * update tests VAN-1599 * Revert "fix: special characters in redirect url getting decoded to space (#1029)" (#1030) This reverts commit fc62241. * feat: update recommendations page design (#1036) VAN-1598 * feat: add events for recommendations (#1039) * feat: remove static recommendations --------- Co-authored-by: Syed Sajjad Hussain Shah <52817156+syedsajjadkazmii@users.noreply.github.com>
* feat: add personalized recommendations (openedx#1024) * use Algolia for personalized recommendations * show personalized recommendations to use that have consented to functional cookies * update tests VAN-1599 * Revert "fix: special characters in redirect url getting decoded to space (openedx#1029)" (openedx#1030) This reverts commit fc62241. * feat: update recommendations page design (openedx#1036) VAN-1598 * feat: add events for recommendations (openedx#1039) * feat: remove static recommendations --------- Co-authored-by: Syed Sajjad Hussain Shah <52817156+syedsajjadkazmii@users.noreply.github.com>
Description
Added events for recommendations shown after registration. Event details can be found in the ticket description.
JIRA
VAN-1600
How Has This Been Tested?