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

feat(ui): show which flag the segment user is using when deleting or updating it #272

Merged
merged 4 commits into from
Jun 26, 2023

Conversation

bimalgrg519
Copy link
Collaborator

@bimalgrg519 bimalgrg519 commented Apr 3, 2023

Fix #276

@cre8ivejp
Copy link
Member

@bimalgrg519, please close and delete the branch if you don't need this PR.

@cre8ivejp cre8ivejp changed the title wip feat(ui): show which flag the segment user is using when deleting or updating it Jun 13, 2023
@bimalgrg519 bimalgrg519 force-pushed the user-segments-users-id branch 3 times, most recently from ac34152 to 8050f5d Compare June 20, 2023 04:25
}),
cannotDelete: defineMessage({
id: 'segment.confirm.delete.cannotDelete',
defaultMessage:
'The 「{segmentName} segment is being used, and you cannot delete it.',
"This {segmentName} segment can't be deleted because {length} {length, plural, one {flag is} other {flags are}} using it.",
Copy link
Member

Choose a reason for hiding this comment

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

nit: This -> The

@bimalgrg519 bimalgrg519 force-pushed the user-segments-users-id branch from 8050f5d to be6a2bf Compare June 24, 2023 11:00
@bimalgrg519 bimalgrg519 force-pushed the user-segments-users-id branch from be6a2bf to e2e70aa Compare June 24, 2023 11:46
@@ -415,6 +415,8 @@
"segment.update.header.description": "ユーザーセグメントを使用すると、単一のフィーチャーフラグバリエーションのすべてのユーザーターゲットを管理できます。これを使用して、多数のユーザーターゲットに対して一度に変更を加えることができます。例えば、既存のベータユーザー群に対して、機能をテスト可能になります。",
"segment.update.header.title": "ユーザーセグメントの更新",
"segment.update.userId": "ユーザーID一覧は他の{length}件のフラグに使用中のため、更新できません。このセグメントをフラグから削除してから更新してください。",
"segment.uploading.message": "ユーザー一覧のアップロードが完了するまでセグメントの更新はできません。",
Copy link
Member

Choose a reason for hiding this comment

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

@bimalgrg519, please update this message as follows.

ユーザー一覧はアップロード中のため、セグメントユーザーの更新ができません。

Copy link
Member

@cre8ivejp cre8ivejp left a comment

Choose a reason for hiding this comment

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

Thank you!

@cre8ivejp cre8ivejp merged commit 0ea2d1a into bucketeer-io:main Jun 26, 2023
Ubisoft-potato pushed a commit to Ubisoft-potato/bucketeer that referenced this pull request Jun 27, 2023
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.

Show which flag is using the Segment User when deleting or updating it
2 participants