-
Notifications
You must be signed in to change notification settings - Fork 18
feat: V2 pro #335
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: V2 pro #335
Conversation
Signed-off-by: Mark Phelps <209477+markphelps@users.noreply.github.com>
Signed-off-by: Mark Phelps <209477+markphelps@users.noreply.github.com>
Signed-off-by: Mark Phelps <209477+markphelps@users.noreply.github.com>
Signed-off-by: Mark Phelps <209477+markphelps@users.noreply.github.com>
There was a problem hiding this 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 pull request introduces comprehensive documentation for Flipt Pro, restructures the navigation, and enhances clarity around licensing and Pro features. The changes focus on improving user experience by making Pro features more discoverable and providing detailed information about enterprise capabilities.
- Adds dedicated Pro features documentation with detailed feature descriptions and pricing
- Restructures navigation hierarchy to improve discoverability with icons and better grouping
- Updates licensing documentation with feature comparison tables and clearer Pro information
Reviewed Changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 9 comments.
Show a summary per file
File | Description |
---|---|
v2/pro.mdx | New comprehensive Pro features page with enterprise capabilities, pricing, and FAQs |
v2/licensing.mdx | Updated with feature comparison table and streamlined Pro licensing information |
v2/introduction.mdx | Minor wording updates and Pro feature reference corrections |
v2/installation.mdx | File rename from overview.mdx, improved link formatting, and layout mode update |
v2/configuration/licensing.mdx | Clarified monthly vs annual licensing connectivity requirements |
docs.json | Navigation restructuring with icons, new Pro Features section, and AI Context addition |
CLAUDE.md | Added development preview instructions for documentation contributors |
v2/pro.mdx
Outdated
GPG signed commits for maximum security and auditability. | ||
</Card> | ||
|
||
{" "} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unnecessary whitespace strings between Card components. These can be removed as the CardGroup component handles spacing automatically.
Copilot uses AI. Check for mistakes.
v2/pro.mdx
Outdated
lock-in, unlimited users, and your data never leaves your environment. | ||
</Card> | ||
|
||
{" "} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unnecessary whitespace strings between Card components. These can be removed as the CardGroup component handles spacing automatically.
Copilot uses AI. Check for mistakes.
v2/pro.mdx
Outdated
keys and configuration data support coming soon. | ||
</Card> | ||
|
||
{" "} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unnecessary whitespace strings between Card components. These can be removed as the CardGroup component handles spacing automatically.
Copilot uses AI. Check for mistakes.
v2/pro.mdx
Outdated
external connectivity required. | ||
</Card> | ||
|
||
{" "} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unnecessary whitespace strings between Card components. These can be removed as the CardGroup component handles spacing automatically.
Copilot uses AI. Check for mistakes.
v2/pro.mdx
Outdated
|
||
<Card | ||
title="Purchase Annual License" | ||
href="<https://getflipt.co/pro/annual>" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
URL has extra angle brackets around it. Should be href="https://getflipt.co/pro/annual"
instead of href="<https://getflipt.co/pro/annual>"
.
href="<https://getflipt.co/pro/annual>" | |
href="https://getflipt.co/pro/annual" |
Copilot uses AI. Check for mistakes.
v2/pro.mdx
Outdated
and dedicated support. A license key is required to access Pro features. | ||
</Accordion> | ||
|
||
{" "} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unnecessary whitespace strings between Accordion components. These can be removed as the AccordionGroup component handles spacing automatically.
Copilot uses AI. Check for mistakes.
v2/pro.mdx
Outdated
on the number of instances you can run with the same license key. | ||
</Accordion> | ||
|
||
{" "} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unnecessary whitespace strings between Accordion components. These can be removed as the AccordionGroup component handles spacing automatically.
Copilot uses AI. Check for mistakes.
v2/pro.mdx
Outdated
include offline validation support, perfect for air-gapped environments. | ||
</Accordion> | ||
|
||
{" "} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unnecessary whitespace strings between Accordion components. These can be removed as the AccordionGroup component handles spacing automatically.
Copilot uses AI. Check for mistakes.
v2/pro.mdx
Outdated
priority bug fixes, and priority consideration for feature requests. | ||
</Accordion> | ||
|
||
{" "} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unnecessary whitespace strings between Accordion components. These can be removed as the AccordionGroup component handles spacing automatically.
Copilot uses AI. Check for mistakes.
Signed-off-by: Mark Phelps <209477+markphelps@users.noreply.github.com>
@claude can you fix these issues brought up by Copilot in the PR review? |
Claude finished @markphelps's task —— View job I've successfully fixed all the Copilot review issues in the Todo List
Changes Made
All Copilot review issues have been resolved. The file now has cleaner markup without unnecessary whitespace strings, and the URL is properly formatted. |
- Remove unnecessary whitespace strings between Card components - Remove unnecessary whitespace strings between Accordion components - Fix URL with extra angle brackets Co-authored-by: Mark Phelps <markphelps@users.noreply.github.com>
This pull request introduces significant updates to Flipt's documentation and user interface, focusing on improving clarity, restructuring content, and adding detailed information about Pro features and licensing. The changes include updates to development commands, restructuring the navigation hierarchy, and introducing comprehensive Pro feature documentation.
Documentation Improvements
mint dev
for previewing documentation and puppeteer MCP tools for browser viewing (CLAUDE.md
).Navigation and Content Restructuring
docs.json
to improve user experience:docs.json
).docs.json
). [1] [2] [3] [4]docs.json
). [1] [2]Pro Features and Licensing
v2/pro.mdx
file detailing Pro features such as enterprise DevOps integration, air-gapped environment support, integrated secrets management, and dedicated support. Includes licensing options, pricing, and FAQs.v2/licensing.mdx
to include a feature comparison table (Free vs Pro), removed outdated purchasing FAQ, and added detailed Pro licensing information. [1] [2]File Renaming and Minor Edits
v2/installation/overview.mdx
renamed tov2/installation.mdx
with updated title and mode (v2/installation.mdx
).v2/installation.mdx
).v2/introduction.mdx
and updated references to Pro features (v2/introduction.mdx
). [1] [2] [3]These changes collectively enhance the usability, clarity, and comprehensiveness of Flipt's documentation, making it easier for users to navigate and understand both free and Pro features.