imprv: Corrected wording on admin page (/admin/data-transfer) #9098

shironegi39 · 2024-09-11T08:38:21Z

Task

改善 #144244 [v7] 管理画面 (/admin/data-transfer) の文言修正
┗ #153642 文言修正

各言語

英語: For more details, please click here.
フランス語: Pour plus de détails, veuillez cliquer ici.
中国語: 有关更多详情，请点击此处。

画面スクリーンショット

org

(左下の path 差分)

cloud

(左下の path 差分)

英語

フランス

中国語

備考

現状、遷移先の「GROWI お引越し機能」のページについては、日本語以外、未実装のためすべての遷移先を、日本語としている、
英語版「GROWI お引越し機能」ページ作成、link の変更については、後続ストーリーを作成

changeset-bot · 2024-09-11T08:38:24Z

⚠️ No Changeset found

Latest commit: ada4bba

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

shironegi39 · 2024-09-12T05:44:02Z

apps/app/public/static/locales/ja_JP/commons.json

@@ -159,6 +159,6 @@
    "publish_transfer_key": "移行キーを発行する",
    "transfer_key_limit": "※ 移行キーの有効期限は発行から1時間となります。",
    "once_transfer_key_used": "※ 移行キーは一度移行に利用するとそれ以降はご利用いただけなくなります。",
-    "transfer_to_growi_cloud": "※ GROWI.cloud への移行を実施する場合はこちらをご確認ください。"
+    "transfer_to_growi_cloud": "※ 詳しくは <a href='https://{{growiDataTransferHelpPage}}/ja/admin-guide/management-cookbook/g2g-transfer.html'>こちら</a>"


https://{{growiDataTransferHelpPage}}/ja/admin-guide/management-cookbook/g2g-transfer.html
として、後続ストーリーで en ページができた際に、 ja 部分を en に変更すると英語ページに遷移できる想定です

growiDataTransferHelpPage という変数名には少し違和感があります

growi.cloud/help or docs.growi.org が来るのであればドメイン名なのでそのニュアンスの変数名に修正お願いします

growiHelpDomain
に変更しました

すみません、growi.cloud/help もあるのでドメインではなかったですね.. documentationUrl などはどうでしょうか？

miya · 2024-09-12T06:24:43Z

apps/app/src/client/components/Admin/G2GDataTransfer.tsx

@@ -123,6 +126,8 @@ const G2GDataTransfer = (): JSX.Element => {
    }
  }, [setTransferring, startTransferKey, selectedCollections, optionsMap]);

+  const growiDataTransferHelpPage = growiCloudUri != null && growiAppIdForGrowiCloud != null ? 'growi.cloud/help' : 'docs.growi.org';
+


このロジックは GROWI 全体で使えそうなロジックなので、useGrowiCloudUri と useGrowiAppIdForGrowiCloud を見て growi.cloud/help か docs.growi.org を返却してくれるカスタム Hook を作ったほうがいいかもしれません。

参考

https://github.com/weseek/growi/blob/e25be0382777e0100735252fb4d8d63288b4026c/apps/app/src/stores-universal/context.tsx#L211-L288

作成しました

miya · 2024-09-12T06:29:34Z

apps/app/public/static/locales/ja_JP/commons.json

@@ -159,6 +159,6 @@
    "publish_transfer_key": "移行キーを発行する",
    "transfer_key_limit": "※ 移行キーの有効期限は発行から1時間となります。",
    "once_transfer_key_used": "※ 移行キーは一度移行に利用するとそれ以降はご利用いただけなくなります。",
-    "transfer_to_growi_cloud": "※ GROWI.cloud への移行を実施する場合はこちらをご確認ください。"
+    "transfer_to_growi_cloud": "※ 詳しくは <a href='https://{{growiDataTransferHelpPage}}/ja/admin-guide/management-cookbook/g2g-transfer.html'>こちら</a>"


growiDataTransferHelpPage という変数名には少し違和感があります

growi.cloud/help or docs.growi.org が来るのであればドメイン名なのでそのニュアンスの変数名に修正お願いします

miya · 2024-09-12T09:59:18Z

apps/app/src/stores-universal/context.tsx

@@ -285,3 +285,18 @@ export const useAcceptedUploadFileType = (): SWRResponse<AcceptedUploadFileType,
    },
  );
 };
+
+export const useGrowiHelpDomain = (): SWRResponse<'growi.cloud/help' | 'docs.growi.org', Error> => {


Help という文字が含まれていると GROWI.cloud の Help サイトっぽいニュアンスになるので useGrowiDocumentationUrl などはどうでしょうか？

useGrowiDocumentationUrl
に変更しました

miya · 2024-09-12T10:01:58Z

apps/app/public/static/locales/ja_JP/commons.json

@@ -159,6 +159,6 @@
    "publish_transfer_key": "移行キーを発行する",
    "transfer_key_limit": "※ 移行キーの有効期限は発行から1時間となります。",
    "once_transfer_key_used": "※ 移行キーは一度移行に利用するとそれ以降はご利用いただけなくなります。",
-    "transfer_to_growi_cloud": "※ GROWI.cloud への移行を実施する場合はこちらをご確認ください。"
+    "transfer_to_growi_cloud": "※ 詳しくは <a href='https://{{growiDataTransferHelpPage}}/ja/admin-guide/management-cookbook/g2g-transfer.html'>こちら</a>"


すみません、growi.cloud/help もあるのでドメインではなかったですね.. documentationUrl などはどうでしょうか？

miya · 2024-09-12T10:04:22Z

apps/app/src/stores-universal/context.tsx

+  const { data: growiAppIdForGrowiCloud } = useGrowiAppIdForGrowiCloud();
+
+  return useSWRImmutable(
+    [growiCloudUri, growiAppIdForGrowiCloud],


これより上に const key = "growiDocumentationUrl" のような key を作って useSWRImmutable の配列に入れるようにしてください

他の Hook を見ると
['documentationUrl', growiCloudUri, growiAppIdForGrowiCloud]
でも行けそうなので、こちらに変更しました

…ected-wording-on-management-screeen" This reverts commit 26b6204, reversing changes made to 42a65bd.

…642-corrected-wording-on-management-screeen"" This reverts commit 5511beb.

JP language support

d281014

github-actions bot added the type/improvement label Sep 11, 2024

shironegi39 changed the title ~~Corrected wording on admin page (/admin/data-transfer)~~ imprv: Corrected wording on admin page (/admin/data-transfer) Sep 11, 2024

shironegi39 added 3 commits September 11, 2024 11:56

Change Variables

5670c27

EN language support

e30356c

Languages Support

7292c56

shironegi39 marked this pull request as ready for review September 12, 2024 05:39

shironegi39 commented Sep 12, 2024

View reviewed changes

miya requested changes Sep 12, 2024

View reviewed changes

shironegi39 added 5 commits September 12, 2024 06:46

Variable Change

f02b84c

Change method to SWRHook

37c6ae6

Change custom hook name

41ff64e

Variable Change

7795fd5

Change custom hook name

ada4bba

miya reviewed Sep 12, 2024

View reviewed changes

miya approved these changes Sep 12, 2024

View reviewed changes

mergify bot added a commit that referenced this pull request Sep 12, 2024

Merge of #9098

057b9c2

mergify bot mentioned this pull request Sep 12, 2024

merge queue: embarking master (42a65bd) and #9098 together #9105

Closed

5 tasks

mergify bot merged commit 26b6204 into master Sep 12, 2024
17 checks passed

mergify bot deleted the imprv/144244-153642-corrected-wording-on-management-screeen branch September 12, 2024 10:10

github-actions bot mentioned this pull request Sep 12, 2024

Release v7.0.19 #9082

Merged

miya added a commit that referenced this pull request Sep 12, 2024

Revert "Merge pull request #9098 from weseek/imprv/144244-153642-corr…

5511beb

…ected-wording-on-management-screeen" This reverts commit 26b6204, reversing changes made to 42a65bd.

miya restored the imprv/144244-153642-corrected-wording-on-management-screeen branch September 12, 2024 10:17

miya added a commit that referenced this pull request Sep 12, 2024

Revert "Revert "Merge pull request #9098 from weseek/imprv/144244-153…

fd28d3b

…642-corrected-wording-on-management-screeen"" This reverts commit 5511beb.

shironegi39 mentioned this pull request Sep 13, 2024

imprv: Corrected wording on admin page (/admin/data-transfer) #9106

Merged

github-actions bot mentioned this pull request Oct 18, 2024

Release v7.0.22 #9274

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

imprv: Corrected wording on admin page (/admin/data-transfer) #9098

imprv: Corrected wording on admin page (/admin/data-transfer) #9098

shironegi39 commented Sep 11, 2024 •

edited

Loading

changeset-bot bot commented Sep 11, 2024 •

edited

Loading

shironegi39 Sep 12, 2024 •

edited

Loading

miya Sep 12, 2024 •

edited

Loading

shironegi39 Sep 12, 2024 •

edited

Loading

miya Sep 12, 2024

miya Sep 12, 2024

shironegi39 Sep 12, 2024

miya Sep 12, 2024 •

edited

Loading

miya Sep 12, 2024

shironegi39 Sep 13, 2024

miya Sep 12, 2024

miya Sep 12, 2024

shironegi39 Sep 13, 2024 •

edited

Loading

imprv: Corrected wording on admin page (/admin/data-transfer) #9098

imprv: Corrected wording on admin page (/admin/data-transfer) #9098

Conversation

shironegi39 commented Sep 11, 2024 • edited Loading

Task

各言語

画面スクリーンショット

org

cloud

英語

フランス

中国語

備考

changeset-bot bot commented Sep 11, 2024 • edited Loading

⚠️ No Changeset found

shironegi39 Sep 12, 2024 • edited Loading

Choose a reason for hiding this comment

miya Sep 12, 2024 • edited Loading

Choose a reason for hiding this comment

shironegi39 Sep 12, 2024 • edited Loading

Choose a reason for hiding this comment

miya Sep 12, 2024

Choose a reason for hiding this comment

miya Sep 12, 2024

Choose a reason for hiding this comment

参考

shironegi39 Sep 12, 2024

Choose a reason for hiding this comment

miya Sep 12, 2024 • edited Loading

Choose a reason for hiding this comment

miya Sep 12, 2024

Choose a reason for hiding this comment

shironegi39 Sep 13, 2024

Choose a reason for hiding this comment

miya Sep 12, 2024

Choose a reason for hiding this comment

miya Sep 12, 2024

Choose a reason for hiding this comment

shironegi39 Sep 13, 2024 • edited Loading

Choose a reason for hiding this comment

shironegi39 commented Sep 11, 2024 •

edited

Loading

changeset-bot bot commented Sep 11, 2024 •

edited

Loading

shironegi39 Sep 12, 2024 •

edited

Loading

miya Sep 12, 2024 •

edited

Loading

shironegi39 Sep 12, 2024 •

edited

Loading

miya Sep 12, 2024 •

edited

Loading

shironegi39 Sep 13, 2024 •

edited

Loading