Drop "shared skill library" from the team invitation email - #171
Conversation
The copy rule active since August 19 forbids "shared library" as a definition of the product. Skills Board is a web app a team signs in to; the library is what the team builds inside it. One string in emails/team-invitation.tsx. The replacement uses the teamName variable already bound in the template and the product's own verb, "keeps". No other change.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Team Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
1 participant
What this is
One string.
emails/team-invitation.tsxtold the invitee to "access the shared skill library", which is the exact formula the copy rule active since August 19 forbids: Skills Board is never defined as a shared library. It is the web app a team signs in to, and the library is what the team builds inside it.Why this wording
teamNameis already bound in the template, so the line can name the actual team instead of a generic object. "Keeps" is the product's own verb, straight from the approved third person phrasing, "the web app where a team keeps and shares its AI skills". The sentence now says what the invitee will find and where, without defining the product as a library and without the word this repo has ruled out.Scope
Nothing else. No component, no props, no styles, no other template.
A sweep of the repository found further user-facing occurrences of "shared skill library" in marketing pages, SEO content, OG image alt text, and app chrome. They are deliberately left alone: they are their own review, and mixing them into this change would bury a one line fix.
Verification
tsc --noEmit: clean.node --test tests/*.test.mjs: 562 of 562 passing.pnpm build: successful production build.No test covers this template, so the change is confirmed by the build and by reading the rendered string, not by a new assertion.