Skip to content

Fix stratifications table not loading on page refresh#2931

Merged
zackcl merged 1 commit intodevfrom
feature/2930-fetch-stratifications-on-refresh
Feb 2, 2026
Merged

Fix stratifications table not loading on page refresh#2931
zackcl merged 1 commit intodevfrom
feature/2930-fetch-stratifications-on-refresh

Conversation

@zackcl
Copy link
Collaborator

@zackcl zackcl commented Feb 2, 2026

Resolves #2930

Problem

Stratifications table was empty after refreshing the Participants page while the Preview Users tab is temporarily disabled.

Solution

  • Changed ngOnInit to call selectedTabChange({index: 1}) to match current tab structure
  • Added sequential data loading: fetch experiment names first, then stratifications after names are loaded (otherwise, an error occurs) using filter and take(1)

Changes

  • experiment-users-root.component.ts: Sequential fetch logic for stratifications tab
  • Child component remains unchanged (subscribe-only pattern)

Revert Instructions

When re-enabling Preview Users tab:

  1. Uncomment tab in HTML
  2. Change index: 1 back to index: 0 in ngOnInit

@zackcl zackcl self-assigned this Feb 2, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes a bug where the stratifications table would not load when refreshing the Participants page while the Preview Users tab is temporarily disabled.

Changes:

  • Updated tab initialization to match current tab structure (index 1 instead of 0)
  • Implemented sequential data loading to ensure experiment names are fetched before stratifications

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@zackcl zackcl merged commit 1c8072d into dev Feb 2, 2026
10 checks passed
@zackcl zackcl deleted the feature/2930-fetch-stratifications-on-refresh branch February 2, 2026 19:25
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.

Refreshing the Participants page clears Stratifications table

2 participants