Skip to content

Conversation

@Kubosaka
Copy link
Collaborator

#913

概要

  • マイページのAPI繋ぎ込み

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

  • URL
    スクリーンショット 2025-02-17 0 32 19

テスト項目

  • DBにデータを追加(financial_records、divisions、user_groups、festival_items、item_budgets、buy_reports、buy_status)
  • ログイン→マイページに移動でデータが表示されるか

備考

@Kubosaka Kubosaka requested a review from TkymHrt February 16, 2025 15:33
@Kubosaka Kubosaka self-assigned this Feb 16, 2025
Copy link
Collaborator

@hikahana hikahana left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@TkymHrt TkymHrt left a comment

Choose a reason for hiding this comment

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

デザインだとサブテーブルにも繰り返し物品名を表示するようだったので、コメントしておきます。冗長なら消すでも良い気もします。

参考までに(92行-123行)

        <TableSubItem
          items={item.buyReports}
          itemsName={item.festivalItemName}
          truncateItemName={truncateItemName}
        />
      )}
    </>
  );
};

// テーブルのサブ項目
interface TableSubItemProps {
  items: BuyReportInformation[];
  itemsName: string | undefined;
  truncateItemName: (value: string, maxLen?: number) => string;
}

const TableSubItem: React.FC<TableSubItemProps> = ({ items, itemsName, truncateItemName }) => {
  return (
    <tr>
      <td colSpan={5}>
        <div className='flex w-full justify-start px-2 md:w-[90%]'>
          <div className='mx-6 w-full overflow-auto rounded-md bg-[#F3F3F3] px-10 py-3'>
            <table
              className='w-full table-auto whitespace-normal break-words text-sm text-[#333]'
              onClick={(e) => e.stopPropagation()}
            >
              <tbody>
                {items.map((item) => (
                  <tr key={item.buyReportName}>
                    <td className='w-[30%] text-nowrap px-2 py-2 text-left'>
                      {truncateItemName(itemsName || '')}

@Kubosaka
Copy link
Collaborator Author

@TkymHrt
修正しますた
自明なので不要な気もしますが、デザインに則りましょう

@Chikuwa0141
Copy link
Collaborator

スクリーンショット 2025-02-24 135109
使用済みの金額を清算完了の(封詰め完了、清算完了チェックが入った)購入報告の金額にお願いします。
画像の景品と食材が清算完了以外の購入報告も含んでいます。

@Kubosaka
Copy link
Collaborator Author

Kubosaka commented Feb 24, 2025

@Chikuwa0141 修正しました
マイページAPI計算修正

@Kubosaka Kubosaka merged commit cfee4bb into feat/yama/913/create-mypage-front Feb 24, 2025
1 check passed
@Kubosaka Kubosaka deleted the feat/kubosaka/mypage-frontend-api branch February 24, 2025 06:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants