Rename workspace role display names from Contributor to Creator#373
Rename workspace role display names from Contributor to Creator#373yousoph wants to merge 1 commit intopreset-io:mainfrom
Conversation
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>
|
|
|
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. |
Vitor-Avila
left a comment
There was a problem hiding this comment.
Requesting some changes to make it backwards compatible
| "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", |
There was a problem hiding this comment.
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.
| "primary contributor": "PresetAlpha", | ||
| "secondary contributor": "PresetBeta", | ||
| "limited contributor": "PresetGamma", |
There was a problem hiding this comment.
But here we should keep these + add the new names. This way the import would support both old and new exports.
|
@yousoph you'll also need to sign the CLA here |
Summary
PresetAlpha,PresetBeta,PresetGamma) remain unchangedTest plan
main_test.pyandexport_users_test.pypass🤖 Generated with Claude Code