Coming back to the unified dashboard from any other screen in the application (confirm, search, editor, ...) always logs the event source as direct.
This is caused by the previousRoute not being stored in the application state in most cases following an unrelated patch. More details here: T388031#10727022
Steps
- Open the unified dashboard
- Click on any translation suggestion
- From the confirm screen, click the close (x) button to go back to the dashboard
Actual
- A "dashboard_open" event is being logged with event_source: direct
Expected
- A "dashboard_open" event is being logged with event_source: return_from_confirmation
Note: there are at least 5 specific event sources for different paths the user can take back to the dashboard. I don't know if re-testing them all is in scope for this task but doing a spot check can't hurt.
Derived Requirement
Ensure that the #dashboard_open event logs the correct 'event_source' based on the user's navigation path, such as 'return_from_confirmation' when returning from the confirmation screen.
Test Steps
Test Case 1: Ensure #dashboard_open event logs 'return_from_confirmation' when navigating back from confirm screen
- Open the Content Translation unified dashboard.
- Click on any available translation suggestion to start a translation.
- On the confirmation screen, click the close (X) button to return to the dashboard.
- Inspect the logged events (via instrumentation or developer tools).
- ✅❓❌⬜ AC1: Confirm that the #dashboard_open event logs 'event_source' as 'return_from_confirmation' instead of 'direct'.
Test Case 2: Spot check that #dashboard_open event source varies appropriately from different navigation paths
- From the dashboard, go to the search screen or editor.
- Navigate back to the dashboard via the standard route (e.g., browser back or in-app UI).
- Inspect the #dashboard_open event in the logs.
- ✅❓❌⬜ AC2: Confirm that the 'event_source' in the #dashboard_open event reflects the appropriate origin (e.g., 'return_from_editor', 'return_from_search') rather than defaulting to 'direct'.
QA Results - Test Beta
| AC | Status | Details |
|---|---|---|
| 1 | ✅ | T391500#10784028 |
| 2 | ✅ | T391500#10784028 |
