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

申請詳細ページをFigmaに合わせる #141

Merged
merged 28 commits into from
Jun 23, 2024
Merged

Conversation

mehm8128
Copy link
Contributor

@mehm8128 mehm8128 commented Apr 26, 2024

コード全部読むのは大変だと思うので、動くかどうかとなんとなくのコードを見てもらえれば多分大丈夫です(あとから直せるし)

  • 上の情報編集
    • タイトル
    • 状態
      • メニュー
        • ステータスチップがクリックできることをわかりやすくする
      • 状態変更モーダル
  • 左の情報編集
    • 詳細情報の編集
  • 右の情報編集
    • 必要に応じてtagの新規作成
    • 対象者
      • 編集
        • 編集して送信できるように
        • なぜか元のデータも書き換えられてしまう
      • 対象者の削除
      • 入出金記録へリンク
  • 権限をいい感じに
  • エラーハンドリング
  • 「入出金記録に紐づいている申請のこの情報を変更すると、入出金記録の情報にも変更が反映されます。よろしいですか?」
    • 本当にこの仕様でいいのか怪しい気してきたから一旦パスで
  • requestがnullになってる
    • useRequestでuseFetchRequestUsecaseを使ってるけど、fetchにuseRequestを使っていないため
  • ファイル周り
    • デザイン決まってないので別でやります

@mehm8128 mehm8128 self-assigned this Apr 26, 2024
@mehm8128 mehm8128 linked an issue May 11, 2024 that may be closed by this pull request
@mehm8128 mehm8128 requested a review from reiroop May 22, 2024 11:45
@mehm8128 mehm8128 marked this pull request as ready for review May 22, 2024 11:46
@mehm8128
Copy link
Contributor Author

大体できたつもりなので一旦レビューリクエスト

@anko9801 anko9801 self-requested a review May 24, 2024 14:34
Copy link
Contributor

@reiroop reiroop left a comment

Choose a reason for hiding this comment

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

コメント書くの忘れてました!すみません
動作とデザインは見ました、いいと思います。コードのほうなんですが、概要掴むのは時間かかりそうです、まださらっと見ただけって感じです。読んでもあまり役には立てないと思うのであんこさんにもレビュー頼みました。ある程度見たらapproveしようと思います。急ぎとかあれば言ってください。

Copy link
Contributor

@reiroop reiroop left a comment

Choose a reason for hiding this comment

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

デザインと基本動作についてのみ確認しました

@mehm8128 mehm8128 merged commit 87eafe4 into main Jun 23, 2024
4 checks passed
@mehm8128 mehm8128 deleted the feat/request_detail branch June 23, 2024 04:32
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.

申請詳細ページ
2 participants