Skip to content

Comments

Rename workspace role display names from Contributor to Creator#373

Open
yousoph wants to merge 1 commit intopreset-io:mainfrom
yousoph:rename-contributor-to-creator
Open

Rename workspace role display names from Contributor to Creator#373
yousoph wants to merge 1 commit intopreset-io:mainfrom
yousoph:rename-contributor-to-creator

Conversation

@yousoph
Copy link
Member

@yousoph yousoph commented Feb 14, 2026

Summary

  • Renames display names for three workspace roles: Primary/Secondary/Limited Contributor → Primary/Secondary/Limited Creator
  • Underlying role identifiers (PresetAlpha, PresetBeta, PresetGamma) remain unchanged
  • Updates both source code and tests

Test plan

  • All 64 existing tests in main_test.py and export_users_test.py pass
  • Verify role display names appear correctly in the UI after import/export operations

🤖 Generated with Claude Code

Rename display names for three workspace roles:
- Primary Contributor -> Primary Creator
- Secondary Contributor -> Secondary Creator
- Limited Contributor -> Limited Creator

The underlying role identifiers (PresetAlpha, PresetBeta, PresetGamma)
remain unchanged.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@sadpandajoe
Copy link

I believe this PR is a breaking change moving forward. They way it shoudl work is support both for now, and then we deprecate the old names later.

Copy link
Contributor

@Vitor-Avila Vitor-Avila left a comment

Choose a reason for hiding this comment

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

Requesting some changes to make it backwards compatible

Comment on lines -386 to -395
"PresetAlpha": "primary contributor",
"PresetBeta": "secondary contributor",
"PresetGamma": "limited contributor",
"PresetAlpha": "primary creator",
"PresetBeta": "secondary creator",
"PresetGamma": "limited creator",
"PresetReportsOnly": "viewer",
"PresetDashboardsOnly": "dashboard viewer",
"PresetNoAccess": "no access",
# Additional possible role identifiers
"Alpha": "primary contributor",
"Beta": "secondary contributor",
"Gamma": "limited contributor",
Copy link
Contributor

Choose a reason for hiding this comment

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

I believe we can keep these changes as-is (new exports will include the updated name). We just need to make sure to only merge this once the change is live.

Comment on lines -92 to -94
"primary contributor": "PresetAlpha",
"secondary contributor": "PresetBeta",
"limited contributor": "PresetGamma",
Copy link
Contributor

Choose a reason for hiding this comment

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

But here we should keep these + add the new names. This way the import would support both old and new exports.

@Vitor-Avila
Copy link
Contributor

@yousoph you'll also need to sign the CLA here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants