docs: Homepage and Getting Started section#5772
Merged
robertnorrie merged 13 commits intoFlagsmith:docs/writechoice-dev-docsfrom Aug 29, 2025
Merged
docs: Homepage and Getting Started section#5772robertnorrie merged 13 commits intoFlagsmith:docs/writechoice-dev-docsfrom
robertnorrie merged 13 commits intoFlagsmith:docs/writechoice-dev-docsfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
|
@guiwrite is attempting to deploy a commit to the Flagsmith Team on Vercel. A member of the Team first needs to authorize it. |
Contributor
There was a problem hiding this comment.
I've added a few specific comments, but I want to also provide some general feedback that might be relevant to the rest of the PRs as I go through them:
- Consistency of capitalisation. Unless a term is a proper noun, we shouldn't capitalise it. I have seen instances where feature, flag, organisation, project, etc. have all been capitalised, and other instance where they are not capitalised. Let's stick to not capitalising.
- Identities are not necessarily users. Please try to avoid indicating that identities are users. In a lot of cases they probably are, but they don't have to be - they can be a device, a tenant, a server, etc. - it's why they're called identities, not users.
Also, related to the PR itself, please make sure that you're following conventional commit guidelines with the title of your PRs - I've updated the title of this one, but please keep in mind for future PRs.
matthewelwell
changed the title
Co-authored-by: Matthew Elwell <mjelwell89@gmail.com>
matthewelwell
requested review from
robertnorrie
and removed request for
Zaimwa9
guiwrite
commented
Aug 22, 2025
robertnorrie
merged commit cb22673
into
Flagsmith:docs/writechoice-dev-docs
5 of 7 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Thanks for submitting a PR! Please check the boxes below:
docs/if required so people know about the feature!Changes
How did you test this code?
No changes to the code, only to the docs.