Skip to content

Feat/796 match user page - #804

Open
vbeni30 wants to merge 3 commits into
devfrom
feat/796-match-user-page
Open

vbeni30 wants to merge 3 commits into
devfrom
feat/796-match-user-page

Conversation

@vbeni30

@vbeni30 vbeni30 commented Aug 21, 2026

Copy link
Copy Markdown

No description provided.

Seme30
Seme30 previously requested changes Aug 31, 2026

@Seme30 Seme30 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Reviewed against issue #796. The default view for another user now opens Report Runs, matching Razor, and the seven navigation targets remain wired. The remaining definition-of-done gap is a rendered frontend test proving that UserPageTabs shows the Report Runs body by default; the current test covers only the tab-selection helper. Please add that rendered test.

@Seme30
Seme30 dismissed their stale review September 18, 2026 14:09

Addressed by the last commit 9b226e9 (9b226e9): added a rendered UserPageTabs test that Report Runs is the default body for another user. Ready for team lead merge.

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.

2 participants