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

Docを初回公開する時のみ「Doc公開」、それ以外は「内容を更新」と表示するようボタンテキストを変更 #8100

Merged
merged 2 commits into from
Oct 13, 2024

Conversation

Ryooo-k
Copy link
Contributor

@Ryooo-k Ryooo-k commented Oct 1, 2024

Issue

概要

Doc作成の際、公開していない状態でWIPとした場合、下図の赤枠部分のボタンテキストが「内容を更新」と表示されているが、公開していない場合は「Docを公開」と表示されるように変更いたします。

スクリーンショット 2024-10-01 15 11 28

変更確認方法

  1. chore/change-button-text-update-content-to-publish-docをローカルに取り込む
    I. git fetch origin pull/8100/head:chore/change-button-text-update-content-to-publish-doc
    Ⅱ. git checkout chore/change-button-text-update-content-to-publish-doc
  2. foreman start -f Procfile.devでローカルサーバーを立ち上げる
  3. 任意のアカウントでログイン
  4. 下記の要領で、ボタンテキストの確認を行う
    I. Doc作成ページを開き、ボタンテキストがDocを公開になっているか確認
    Ⅱ. データ入力後WIPボタンを押しても、ボタンテキストがDocを公開のままになっているか確認
    Ⅲ. Docを公開ボタンを押した後、内容変更ボタンを押し編集画面に移動する。そして、ボタンテキストが内容を更新になっているか確認
    Ⅳ. WIPボタンを押す。公開後、WIP状態にしてもボタンテキストが内容を更新になっているか確認

Screenshot

変更前

スクリーンショット 2024-10-01 15 11 28

変更後

スクリーンショット 2024-10-01 15 15 28

@Ryooo-k Ryooo-k self-assigned this Oct 1, 2024
@Ryooo-k Ryooo-k marked this pull request as ready for review October 1, 2024 07:12
@Ryooo-k
Copy link
Contributor Author

Ryooo-k commented Oct 1, 2024

@kurumadaisuke
お疲れ様です!こちらのレビューお願いしたいのですが可能でしょうか?🙇
ご確認お願いいたします。

@kurumadaisuke
Copy link
Contributor

@Ryooo-k
はい!確認させていただきます🙏
少々お時間いただければと思います🙏

@kurumadaisuke
Copy link
Contributor

@Ryooo-k
挙動について1点確認させてください!!

・Doc作成ページを開き、適当に「タイトル」「本文」にデータ入力後WIPボタンを押し、ボタンテキストが「Docを公開」になっているか確認
っであっていますでしょうか?
ローカルで確認しますと「内容を更新」っとなっていたのでこちらは挙動としてあっているでしょうか🙏
スクリーンショット 2024-10-07 20 38 31

@Ryooo-k
Copy link
Contributor Author

Ryooo-k commented Oct 7, 2024

@kurumadaisuke
ご連絡ありがとうございます!

Doc作成ページを開き、適当に「タイトル」「本文」にデータ入力後WIPボタンを押し、ボタンテキストが「Docを公開」になっているか確認
っであっていますでしょうか?

挙動の確認は合っています👌

ローカルで確認しますと「内容を更新」っとなっていたのでこちらは挙動としてあっているでしょうか🙏

下記動画の通り私の環境だと、新規作成→データ入力→WIP→「Docを公開」になっているんですよね💦

2024-10-07.23.50.47.mov

お手数おかけしますが、動画で共有いただけないでしょうか🙏

Copy link
Contributor

@kurumadaisuke kurumadaisuke left a comment

Choose a reason for hiding this comment

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

すいません、自分が確認手順間違っていました🙏
挙動あっていましたのでApproveいたします!

@Ryooo-k
Copy link
Contributor Author

Ryooo-k commented Oct 8, 2024

@kurumadaisuke
ご確認ありがとうございます!大変申し訳ありませんが、修正漏れがありました💦
再度ご確認いただけないでしょうか🙇💦

確認内容

  • 通知メッセージの確認
    • 「Docを公開」を押した場合は、通知メッセージが「ドキュメントを作成しました。」と表示される
    • 「内容を更新」を押した場合は、通知メッセージが「ドキュメントを更新しました。」と表示される

現状

現状は下記動画のように「Docを公開」を押した際、右上に緑色の通知メッセージが表示されますが、そのメッセージ内容が「ドキュメントを更新しました。」と表示されてしまっていますので、今回の修正に合わせた変更となります。

default.mov

修正内容

3c6ce59

@kurumadaisuke
Copy link
Contributor

@Ryooo-k
確認不足ですいません🙏🙏
確認できましたのでOKです!!

@Ryooo-k
Copy link
Contributor Author

Ryooo-k commented Oct 10, 2024

@kurumadaisuke
ご確認ありがとうございます!何度も見ていただきありがとうございました✨

@komagata
お疲れ様です!
メンバーからApproveいただきましたのでレビューお願いいたします。

@Ryooo-k Ryooo-k requested a review from komagata October 10, 2024 01:06
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 7d94c37 into main Oct 13, 2024
4 checks passed
@komagata komagata deleted the chore/change-button-text-update-content-to-publish-doc branch October 13, 2024 22:36
@github-actions github-actions bot mentioned this pull request Oct 13, 2024
7 tasks
@Ryooo-k
Copy link
Contributor Author

Ryooo-k commented Oct 14, 2024

ご確認ありがとうございます!

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