Skip to content

[gm3] お知らせ欄の作成 #1786

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.

Sign up for GitHub

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

Jump to bottom
Merged
harata-t merged 12 commits into from
Apr 11, 2025
Merged

[gm3] お知らせ欄の作成 #1786

harata-t merged 12 commits into from
Apr 11, 2025

Conversation

@hikahana
Copy link
Contributor

@hikahana hikahana commented Mar 24, 2025 •
edited by izuizu0424
Loading

対応Issue

resolve #1777

概要

  • お知らせ欄を作成しました。

実装詳細

  • created_atを日付として取得しています。
  • そこでdate-fnsというライブラリを導入しました。

画面スクリーンショット等

image

テスト項目

  • デザインと差異がないか
  • お知らせの値が取得できているか
  • [ ]

備考

api繋ぎ込みをgroupIDで取得しているが、団体によってメッセージが異なるとかあるのですか？

@hikahana hikahana requested review from harata-t and izuizu0424 March 24, 2025 14:32
@hikahana hikahana self-assigned this Mar 24, 2025
izuizu0424
izuizu0424 reviewed Mar 26, 2025
View reviewed changes
Copy link
Collaborator

@izuizu0424 izuizu0424 left a comment

Choose a reason for hiding this comment

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

[デザインについて]
お知らせモーダルのみパディングが小さくなっていたため，Figmaデザインを変更しました．
スタイルコンポーネントのページに追加したので確認お願いします．

追記：ログイン画面では幅の異なるお知らせモーダルがあったんですが，これは一つのコンポーネントで変更可能ですか？
image

@hikahana
Copy link
Contributor Author

hikahana commented Mar 26, 2025

追記：ログイン画面では幅の異なるお知らせモーダルがあったんですが，これは一つのコンポーネントで変更可能ですか？

よく分からないけどログイン画面と申請画面では大きさが異なるってことで良いのよね。
対応としては可能です。propsでisLoginみたいなの作成してtrueなら幅増やすみたいな作りにしたら良さそう

izuizu0424
izuizu0424 reviewed Mar 26, 2025
View reviewed changes
Copy link
Collaborator

@izuizu0424 izuizu0424 left a comment

Choose a reason for hiding this comment

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

テスト項目「お知らせの値が取得できているか」
なんですが，管理者画面でお知らせを追加したところstorybook上では確認できませんでした．
↓管理者画面
image

↓storybook
image

変更よろしくお願いします．
それとも新しく実行環境を作って確認した方がいい感じですか？

@hikahana
Copy link
Contributor Author

hikahana commented Mar 31, 2025

テスト項目「お知らせの値が取得できているか」
なんですが，管理者画面でお知らせを追加したところstorybook上では確認できませんでした．

これどう追加されているの？
api繋ぎ込みをgroupIDで取得しているが、団体によってメッセージが異なるとかあるのですか？にも聞いている通り、いまgroup_id=1で取得しているんだけど各団体に向けてのお知らせとかするの？それとも全員共通？？？

@izuizu0424
Copy link
Collaborator

izuizu0424 commented Apr 1, 2025

なるほど理解がおよんでいませんでした
聞いてみたところ団体による差はないようなので，全員共通のお知らせが表示されるようにお願いします！

@harata-t
Copy link
Collaborator

harata-t commented Apr 3, 2025

@hikahana
GM2においてお知らせ欄はnewsになってるー
notifyじゃないね、、、

apiもannnouncementsAPIってなってるけど、Newsにしてほしいかも
中身もannouncementsじゃなくてnewsのやつ見て書き換えお願いします！

@hikahana hikahana requested a review from izuizu0424 April 4, 2025 13:29
izuizu0424
izuizu0424 reviewed Apr 6, 2025
View reviewed changes
Copy link
Collaborator

@izuizu0424 izuizu0424 left a comment

Choose a reason for hiding this comment

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

値の取得できてました！ありがとうございます
デザインも前から変わっていないならOKだと思います

harata-t
harata-t approved these changes Apr 11, 2025
View reviewed changes
Copy link
Collaborator

@harata-t harata-t left a comment

Choose a reason for hiding this comment

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

LGTM!

@harata-t harata-t merged commit d3f759e into gm3/develop Apr 11, 2025
1 check passed
@harata-t harata-t deleted the feat/hikahana/1777-add-componet-notify-litt branch April 11, 2025 08:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@izuizu0424 izuizu0424 izuizu0424 left review comments

@harata-t harata-t harata-t approved these changes

Assignees

@hikahana hikahana

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

お知らせ欄の作成

4 participants

@hikahana @izuizu0424 @harata-t