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

お知らせ作成フォームのタイトル部分に説明文を追加した #7469

Merged
merged 3 commits into from
Mar 9, 2024

Conversation

mousu-a
Copy link
Contributor

@mousu-a mousu-a commented Mar 1, 2024

Issue

概要

お知らせ作成フォームのタイトル下に説明文を追加しました。

変更確認方法

  1. feature/added-description-to-the-create-notice-formをローカルに取り込む
  2. foreman start -f Procfile.devでローカルでアプリを起動する
  3. komagata(testtest)(ユーザーは誰でも良いです)でログインする
  4. localhost:3000/announcements/newにアクセスする
  5. タイトルバーの下に注釈が表示されていることを確認する。

Screenshot

変更前

image

変更後

貼り付けた画像_2024_03_02_13_06

@mousu-a mousu-a assigned mousu-a and machida and unassigned machida Mar 1, 2024
@mousu-a mousu-a marked this pull request as ready for review March 3, 2024 03:41
@mousu-a mousu-a requested a review from hirano-vm4 March 3, 2024 03:41
@mousu-a
Copy link
Contributor Author

mousu-a commented Mar 3, 2024

@hirano-vm4
お疲れ様です🙏
お手数おかけしますがこちらレビューいただきたいですー!
お忙しいようであれば遠慮なくおっしゃってください🙇‍♂️
全然急ぎではないです!

@hirano-vm4
Copy link
Contributor

@mousu-a
お疲れ様です!レビュー依頼ありがとうございます😊
今日の夜までには確認してお返しします💪よろしくお願いします!

Copy link
Contributor

@hirano-vm4 hirano-vm4 left a comment

Choose a reason for hiding this comment

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

お疲れ様です!コードを確認させていただきました!

LGTMです〜🙆注意書きによって入力者に意識する点をうまく伝えられるページになりますね💪

@mousu-a
Copy link
Contributor Author

mousu-a commented Mar 3, 2024

@komagata
メンバーにレビューいただけましたので、こちらお手すきの際にレビューお願い致します🙏

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 1c54004 into main Mar 9, 2024
6 checks passed
@komagata komagata deleted the feature/added-description-to-the-create-notice-form branch March 9, 2024 13:37
@github-actions github-actions bot mentioned this pull request Mar 9, 2024
29 tasks
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.

4 participants