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

add /utol/notification #1231

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

add /utol/notification #1231

wants to merge 7 commits into from

Conversation

haxibami
Copy link
Member

No description provided.

@haxibami haxibami added the utol label Oct 18, 2024
@haxibami haxibami changed the title add /utol/notification add /utol/notification Oct 18, 2024
Copy link

github-actions bot commented Oct 18, 2024

🚀 Deployed on https://deploy-preview-1231--utelecon.netlify.app

@haxibami haxibami marked this pull request as ready for review October 22, 2024 05:31
src/pages/utol/notification/index.mdx Show resolved Hide resolved
src/pages/utol/notification/index.mdx Outdated Show resolved Hide resolved
src/pages/utol/notification/index.mdx Outdated Show resolved Hide resolved
src/pages/utol/notification/index.mdx Outdated Show resolved Hide resolved
Comment on lines 139 to 142
1. 以下の,ワークスペース「UTOL 通知」へのリンクを開いてください.
<b class="box center"><a href="https://utol-dm.slack.com/">UTOL 通知</a></b>
2. 既にUTokyo Slackにサインイン済みの場合を除き,UTokyo Slackへのサインイン画面が表示されるので,「UTokyo Accountでサインインする」を押してください.
3. 既にUTokyo Accountにサインイン済みの場合を除き,UTokyo Accountのサインイン画面が表示されるので,サインインしてください.
Copy link
Member

Choose a reason for hiding this comment

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

作業の意味として,2.と3.をもう一段掘ったほうが良いかもしれません.

Copy link
Member Author

Choose a reason for hiding this comment

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

この周辺の記述について:

  • 改めてワークスペース「UTOL 通知」への参加の手順をなぞったところ、文面に比して複雑な手順(リンクを踏んだだけでは済まない)であることが判明したため、画像を含めて詳細に記載しました。
  • また、これに伴って一連の手順が長くなったため、2. 3. を一段深くする代わりに、手順1(「UTOL 通知」への参加)と手順2(UTOL上での設定)を分割しました。
  • /slack/join/#手順 を参考にSlack関連のヘルプ・記述を追加しました。

Copy link
Member

Choose a reason for hiding this comment

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

リンクに飛ぶだけでは参加できなかったのですね…
追記ありがとうございます.

/slack/joinにリンクを飛ばす,あるいは(一部でも)共通化することも考えたほうが良いかもしれません,
特にトラブルシューティングが重複しているのが気になっています(将来的に変わる可能性があるため).

Copy link
Member Author

Choose a reason for hiding this comment

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

指摘ありがとうございます。

たしかに現状では記載が冗長なので、/slack/join にリンクを飛ばして、「UTokyo Slackのワークスペース」の画面を開く段階まで(つまり、サインインなどの作業)はそちらを参照してもらう形式にしようかと思います。

また、この記事のスコープを外れますが、トラブルシューティングについてはUTokyo Slack関連の記事全体で共通化してもよいかもしれません。たとえばUTokyo VPN関連の記事では共通化されています(例1例2)が、現状、UTokyo Slack関連の記事ではバラバラに記載されています。こちらに関しては別PRで作業したほうがよいでしょうか?

Copy link
Member Author

@haxibami haxibami Oct 30, 2024

Choose a reason for hiding this comment

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

ところで、「UTOL 通知」ワークスペースのリンクを踏んだだけでは参加できない(いずれにせよ最終的にはワークスペースを検索する必要がある)以上、utol-dm.slack.com のリンクを記載する意味はなく、/slack/join#手順 に記載の手順に完全に乗ってしまったほうがよいのではないかと思いました。

こちら で触れていましたね、失礼しました)

Copy link
Member Author

Choose a reason for hiding this comment

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

この方針に沿って文面を修正しました。

Copy link
Member

Choose a reason for hiding this comment

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

これで良いかなと思います.強いて言えば「#手順」は書かずに「〜」の手順の5.までを参考に,のほうがしっくりくるかと思いましたが,個人の好みかもしれないので良いです.

src/pages/utol/notification/index.mdx Outdated Show resolved Hide resolved
src/pages/utol/notification/index.mdx Outdated Show resolved Hide resolved
haxibami and others added 2 commits October 28, 2024 15:43
Co-authored-by: phenylshima <49227365+femshima@users.noreply.github.com>
src/pages/utol/notification/index.mdx Outdated Show resolved Hide resolved
Comment on lines 139 to 142
1. 以下の,ワークスペース「UTOL 通知」へのリンクを開いてください.
<b class="box center"><a href="https://utol-dm.slack.com/">UTOL 通知</a></b>
2. 既にUTokyo Slackにサインイン済みの場合を除き,UTokyo Slackへのサインイン画面が表示されるので,「UTokyo Accountでサインインする」を押してください.
3. 既にUTokyo Accountにサインイン済みの場合を除き,UTokyo Accountのサインイン画面が表示されるので,サインインしてください.
Copy link
Member

Choose a reason for hiding this comment

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

リンクに飛ぶだけでは参加できなかったのですね…
追記ありがとうございます.

/slack/joinにリンクを飛ばす,あるいは(一部でも)共通化することも考えたほうが良いかもしれません,
特にトラブルシューティングが重複しているのが気になっています(将来的に変わる可能性があるため).

src/pages/utol/notification/index.mdx Show resolved Hide resolved
src/pages/utol/notification/index.mdx Outdated Show resolved Hide resolved
src/pages/utol/notification/index.mdx Outdated Show resolved Hide resolved
haxibami and others added 2 commits October 30, 2024 13:45
Co-authored-by: phenylshima <49227365+femshima@users.noreply.github.com>
Co-authored-by: phenylshima <49227365+femshima@users.noreply.github.com>
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