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

[相談部屋]未返信タブのときのタイトルを変更 #4184 #4356

Merged
merged 2 commits into from
Mar 14, 2022

Conversation

taka110-IT
Copy link
Contributor

@taka110-IT taka110-IT commented Mar 6, 2022

issue #4184

  • adminでログインし、相談部屋の未返信タブを開いた時のタイトル「未返信のコメント」を「相談部屋」に修正しました。

変更前

#4184作業前

変更後

#4184作業後

ローカルでの確認方法

  • feature/change-talks-unreplied-page-title ブランチをローカル環境で起動する。
  • adminでログインする。
  • 左下の「相談」から「未返信」タブをクリックする。

@taka110-IT taka110-IT self-assigned this Mar 6, 2022
@taka110-IT taka110-IT marked this pull request as draft March 6, 2022 14:56
@taka110-IT taka110-IT marked this pull request as ready for review March 6, 2022 14:59
@taka110-IT taka110-IT requested a review from aim2bpg March 6, 2022 15:01
Copy link
Contributor

@aim2bpg aim2bpg left a comment

Choose a reason for hiding this comment

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

@taka110-IT さん、チーム開発の初Issueおめでとうございます〜🎉🎉🎉
私はOKです🙆‍♀️ チームリーダーレビューに進められてください👍

次回は下記の2点があると、なお良いと思いました〜(他の方のPRを参考に)

  • 変更前後のスクショに下線または枠で変更箇所を明示する。
    リリースノートに載せる際には、たくさんの方が見られますので入れておかれてください。
  • PRコメントと別に、単独コメントでレビュー依頼する。
    PRコメントは色々書きますし、作成途中の場合が多いので単独で入れておかれると、レビュー依頼されたのだなということが、分かりやすいです。

以上は、今後の開発において相手目線で考える練習も兼ねていると思いますので、徐々に慣れていかれれば良いと思います(私もあまりできていないのですが...😅 )

@taka110-IT
Copy link
Contributor Author

@aim2bpg さん、レビューありがとうございました。
見やすく、わかりやすくは何事も基本ですね😅PRコメントを修正しました。
マナーとしてレビュー依頼ボタンを押すだけだと素っ気ない感じだったので、今後は単独コメントを入れるようにします。

@komagata さん、チームリーダーのレビューをお願いします。

@taka110-IT taka110-IT requested a review from komagata March 7, 2022 20:36
@@ -1,4 +1,4 @@
- title '未返信のコメント'
- title '相談部屋'
Copy link
Member

Choose a reason for hiding this comment

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

これだとHTMLのMETA TITLEまで変わってしまわないですかね?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@komagata さん
1行目を元に戻し、h2タグの部分を修正しました。再確認をお願いします。

@taka110-IT taka110-IT force-pushed the feature/change-talks-unreplied-page-title branch from 9ff9b01 to 6b17b18 Compare March 11, 2022 20:35
Copy link
Member

@komagata komagata left a comment

Choose a reason for hiding this comment

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

確認しました、OKですー🙆‍♂️

@komagata komagata merged commit 31cecf7 into main Mar 14, 2022
@komagata komagata deleted the feature/change-talks-unreplied-page-title branch March 14, 2022 14:42
@github-actions github-actions bot mentioned this pull request Mar 14, 2022
32 tasks
@taka110-IT
Copy link
Contributor Author

@komagata さん
確認ありがとうございました。ステージング環境でも確認できました。

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.

3 participants