Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: init hey pro #3985

Merged
merged 3 commits into from Nov 12, 2023
Merged

feat: init hey pro #3985

merged 3 commits into from Nov 12, 2023

Conversation

bigint
Copy link
Member

@bigint bigint commented Nov 12, 2023

What does this PR do?

🤖 Generated by Copilot at 022709c

This pull request adds a new UI component and a feature flag for the Pro feature. It also updates some existing components to include the Pro component in the navbar and the mobile drawer menu, depending on the feature flag.

Related issues

Fixes # (issue)

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Enhancement (non-breaking small changes to existing functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Explanation of the changes

🤖 Generated by Copilot at 022709c

  • Add a new Pro component that renders a link to the /pro page with a briefcase icon and the text "Pro" (link)
  • Import and conditionally render the Pro component in the MobileDrawerMenu component, which is a sidebar menu for mobile devices (MobileDrawerMenu.tsx, link, link)
  • Import and conditionally render the Pro component in the SignedUser component, which is a dropdown menu for signed-in users (SignedUser.tsx, link, link)
  • Add a new feature flag ProImplementation to the FeatureFlag enum, which is used to toggle the visibility of the Pro component (feature-flags.ts, link)
  • Add a comment to the feature-flags.ts file to indicate that the enum values above the comment are static flags, meaning that they are hardcoded in the codebase (link)

Emoji

📄📱🚀

Copy link

height bot commented Nov 12, 2023

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

@bigint bigint self-assigned this Nov 12, 2023
Copy link

vercel bot commented Nov 12, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
prerender ✅ Ready (Inspect) Visit Preview Nov 12, 2023 4:21pm
web ✅ Ready (Inspect) Visit Preview Nov 12, 2023 4:21pm

@vercel vercel bot temporarily deployed to Preview – prerender November 12, 2023 16:10 Inactive
@bigint bigint merged commit 766f96d into main Nov 12, 2023
4 of 7 checks passed
@bigint bigint deleted the pro-init branch November 12, 2023 16:14
@vercel vercel bot temporarily deployed to Preview – prerender November 12, 2023 16:14 Inactive
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.

None yet

1 participant