docs: FAQ page components and data#6402
Conversation
|
@TyroneOrrego is attempting to deploy a commit to the Flagsmith Team on Vercel. A member of the Team first needs to authorize it. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
There was a problem hiding this comment.
This PR is being reviewed by Cursor Bugbot
Details
Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.
To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.
|
@TyroneOrrego can you fix the broken link? You should be able to replicate this error message yourself by running |
Hi @matthewelwell I think I have already fixed it. I just committed the changes! |
|
Hi @TyroneOrrego , thanks for fixing the linting issue. From an aesthetic point of view, I think it looks fairly good, although it still feels a bit 'documentation' vs FAQ. I'm also a little concerned about the manual expectations here - the 'counts' that appear in the tile view need to be updated manually if a new item is added. Could we look at something like this which might help us out? If not, I vote for removing the counts for now. |
|
Hi @matthewelwell, Thanks for the review! I totally agree regarding the manual counts—that’s definitely a maintenance headache waiting to happen. I’m going to look into the solution you linked to see if I can get it working dynamically without too much overhead. Regarding the "documentation" feel, I’ve made some styling changes to make the experience feel a bit lighter. I'll push a commit with those updates included shortly! Cheers, Tyrone |
matthewelwell
left a comment
There was a problem hiding this comment.
Thanks for the aesthetic changes @TyroneOrrego , they look great.
Since we are waiting on these changes before deprecating our old support tool which hosts help.flagsmith.com I'm approving this with a view to merge it, and we can iterate on the code with regards to the changes suggested in my previous comment here in a separate PR.
Thanks for submitting a PR! Please check the boxes below:
docs/if required so people know about the feature!Changes
FAQ section added.
How did you test this code?
Please describe.