Skip to content

feat: introduce learn v2 behind a feature-flag #558

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

GuiBibeau
Copy link

@GuiBibeau GuiBibeau commented Jun 16, 2025

Problem

The current learn section is dull and offers not many informations to learners. This introduces behind a feature flag a new learn v2 section.

Summary of Changes

  • introduce a feature flag that when turned off redirects to /learn allowing us to work in multiple incremental PRs
  • proposes a layout for the learn section.
  • The styling is not final as this PR is meant to be a first stab at the section.
  • The content is full of place holders. Real posts will go there. This pr is not meant to discuss the content.
demo-learn-v2.mp4

Fixes #

Copy link

vercel bot commented Jun 16, 2025

@GuiBibeau is attempting to deploy a commit to the Solana Foundation Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

vercel bot commented Jun 16, 2025

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

Name Status Preview Updated (UTC)
solana-com ✅ Ready (Inspect) Visit Preview Jun 16, 2025 1:41pm

# Learn V2 Development
# Set LOCAL_LEARN_V2=true to enable the learn-v2 feature locally
LOCAL_LEARN_V2=false
Copy link
Collaborator

Choose a reason for hiding this comment

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

We can remove this. Will figure out a long term strategy for FF later.

@@ -25,6 +25,7 @@
"@sindresorhus/slugify": "^2.2.1",
"@solana-foundation/solana-lib": "^2.40.3",
"@typeform/embed": "^5.1.0",
"@vercel/edge-config": "^1.4.0",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Remove along with FF

Copy link
Collaborator

Choose a reason for hiding this comment

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

Use /learn for route and update components to use existing component libs wherever possible rather than custom code to benefit from future upgrade paths.

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