Skip to content

Conversation

@ayesha-waris
Copy link
Contributor

INF-1893
Description

Discussions activity shows a significant decline this year (1st screenshot). Part of the reason could be decline in daily active users which is why i plotted % of DAUs who interact with discussions (2nd screenshot). But that also gone down since 2023 even though we now have tray and email notifications.

I suspect that certificate preview in sidebar has contributed in this decline. The preview pushes discussions below the scroll line (3rd and 4th screenshots).

But this is just a hypothesis! I want to test this hypothesis but reversing the order of the CTA and discussions (discussions on top) for a few days. This means that users will see discussions on top ONLY on units where discussions is enabled. I want to see the impact on discussions activity and purchase.

Deploy changes on Monday and revert on the following Monday. That should give us 7 days of test data.

@ayesha-waris ayesha-waris requested a review from awais-ansari May 12, 2025 13:29
@codecov
Copy link

codecov bot commented May 12, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.41%. Comparing base (b1ee8a3) to head (0938930).
Report is 6 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1705   +/-   ##
=======================================
  Coverage   90.41%   90.41%           
=======================================
  Files         343      343           
  Lines        5779     5779           
  Branches     1346     1346           
=======================================
  Hits         5225     5225           
  Misses        537      537           
  Partials       17       17           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ayesha-waris ayesha-waris requested a review from a team May 13, 2025 08:35
@nsprenkle nsprenkle merged commit 2ce8333 into openedx:master May 19, 2025
7 checks passed
jciasenza pushed a commit to jciasenza/frontend-app-learning that referenced this pull request May 19, 2025
…dx#1705)

Co-authored-by: Ayesha Waris <ayesha.waris@192.168.10.27>
ayesha-waris added a commit to ayesha-waris/frontend-app-learning that referenced this pull request May 21, 2025
nsprenkle pushed a commit that referenced this pull request Jun 5, 2025
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.

4 participants