Skip to content

DEVPROD-32609: Add GuideCue for updated Project navbar#1626

Merged
sophiayangDB merged 4 commits intoevergreen-ci:mainfrom
sophiayangDB:DEVPROD-32609
May 1, 2026
Merged

DEVPROD-32609: Add GuideCue for updated Project navbar#1626
sophiayangDB merged 4 commits intoevergreen-ci:mainfrom
sophiayangDB:DEVPROD-32609

Conversation

@sophiayangDB
Copy link
Copy Markdown
Contributor

DEVPROD-32609

Description

Since we have chosen to add a collapsible section for the new GitHub tabs, this PR introduces a GuideCue so that users are notified of the changes.

Screenshots

Screenshot 2026-05-01 at 10 23 56 AM

Testing

None

@sophiayangDB sophiayangDB requested a review from a team as a code owner May 1, 2026 14:27
<GuideCue
currentStep={1}
data-cy="github-nav-guide-cue"
enabled
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.

it's already blocked in the parent but can we be extra safe and block it here too?

Suggested change
enabled
enabled={showNewProjectNavigation}

return (
<GuideCue
currentStep={1}
data-cy="github-nav-guide-cue"
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.

if there's no tests for it, there's no need for data-cy

Suggested change
data-cy="github-nav-guide-cue"

@@ -0,0 +1,42 @@
// GithubNavGuideCue.tsx
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.

Suggested change
// GithubNavGuideCue.tsx

import { useState } from "react";
import {
GuideCue,
type GuideCueProps,
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.

Suggested change
type GuideCueProps,
GuideCueProps,

@sophiayangDB sophiayangDB merged commit 04627f2 into evergreen-ci:main May 1, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants