Skip to content
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

Change "My Team" to "My Personal Sites" and expose it in sites list #5132

Draft
wants to merge 18 commits into
base: master
Choose a base branch
from

Conversation

zoldar
Copy link
Contributor

@zoldar zoldar commented Mar 3, 2025

Changes

This PR renames "My Team" to "My Personal Sites" when team's setup is not complete yet.

It also enforces team name change during setup, treating "My Personal Sites" as reserved name. There's a default generated on the basis of user's name.

The CTA is also rephrased to "Setup Team" instead of "Create Team" to express the intent better.

The team name is now used in place of "My Sites" heading in sites list view to clearly identify which team the user is switched to. The "My Personal Sites" heading is also the default for a case where user is not a member of any team yet.

The heading change is hidden behind a flag for now.

The migration renaming the teams will be extracted and deployed independently after the PR is approved.

Tests

  • Automated tests have been added

@zoldar zoldar added the preview label Mar 3, 2025
Copy link

github-actions bot commented Mar 3, 2025

Preview environment👷🏼‍♀️🏗️
PR-5132

if site.team.name == "My Team" do
owner.name
else
if site.team.setup_complete do
Copy link
Member

Choose a reason for hiding this comment

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

👌

@@ -12,6 +12,14 @@ defmodule Plausible.Teams do

@accept_traffic_until_free ~D[2135-01-01]

@spec default_name() :: String.t()
def default_name(), do: "My Personal Sites"
Copy link
Member

Choose a reason for hiding this comment

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

👌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants