2026/04/29 mainマージ#1087
Conversation
…seeding strategy updates
…modal, table, and field row
…als for document creation
…r better visibility
…oved alignment and visibility
…lculations and invoice generation
…ne form default values in ProgressReportPage
…ssReport components
…ts for improved clarity and consistency
… in ProgressReport components
…se types in generated hooks
…ons/SponsorActivitiesDesktopSection.tsx Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
- SponsorActivitiesDesktopSection.tsx: リファクタリングの残骸により壊れていたJSXを削除 - globals.css: 産学局用の --color-industry-university テーマ変数を追加 - createPurchaseOrderPdf.ts / createPurchaseReportPdf.ts: BUREAUS を id で検索するよう修正し、bureauID=7 でのクラッシュとオフバイワンを解消 Co-Authored-By: TkymHrt <23.h.takayama.nutfes@gmail.com>
ネイティブのtitle属性に戻すことで、オーバーフロー境界で切り取られる 問題を回避 Co-Authored-By: TkymHrt <23.h.takayama.nutfes@gmail.com>
- min-w-245は無効なTailwindクラスのためmin-w-[980px]に戻す - 産学局用の--color-industry-university CSS変数をglobals.cssに追加し、BureauLabelのスタイル崩れを防止 Co-Authored-By: TkymHrt <23.h.takayama.nutfes@gmail.com>
BUREAUS[bureauID]は配列インデックス(0始まり)としてDBのbureauID(1始まり)を使用しており、 新規追加した産学局(id=7)でundefinedとなりクラッシュするバグを修正。 BUREAUS.find((b) => b.id === bureauID)?.name のパターンに変更。 Co-Authored-By: TkymHrt <23.h.takayama.nutfes@gmail.com>
産学局を追加し、スポンサー活動の表示を改善
…eport-frontend 協賛活動の進捗報告ページのフロントエンド
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces a new "Progress Report" (進捗報告) feature, including a dedicated page and several components for managing sponsorship activity status and generating invoice/receipt PDFs. It also restructures the database schema for buildings and rooms, adds the "Industry-University Bureau" (産学局) across the system, and updates the OpenAPI specification. Feedback focuses on correcting a Tailwind CSS typo, improving error handling for PDF generation, and refactoring hardcoded labels and icon logic to use centralized constants or mappings.
| className='min-w-52 text-md justify-center border-[#56daff] px-4 py-1.5 font-normal text-[#56daff]' | ||
| onClick={() => setIsBlankReceiptModalOpen(true)} | ||
| > | ||
| 手書きで領収書発行 | ||
| </OutlinePrimaryButton> | ||
| <OutlinePrimaryButton | ||
| className='min-w-52 text-md justify-center border-[#56daff] px-4 py-1.5 font-normal text-[#56daff]' |
There was a problem hiding this comment.
Tailwind CSSの標準クラスに text-md は存在しません。text-base または text-sm のタイポである可能性があります。他のコンポーネントとの一貫性を保つため、text-base への修正を検討してください。
| className='min-w-52 text-md justify-center border-[#56daff] px-4 py-1.5 font-normal text-[#56daff]' | |
| onClick={() => setIsBlankReceiptModalOpen(true)} | |
| > | |
| 手書きで領収書発行 | |
| </OutlinePrimaryButton> | |
| <OutlinePrimaryButton | |
| className='min-w-52 text-md justify-center border-[#56daff] px-4 py-1.5 font-normal text-[#56daff]' | |
| className='min-w-52 text-base justify-center border-[#56daff] px-4 py-1.5 font-normal text-[#56daff]' | |
| onClick={() => setIsBlankReceiptModalOpen(true)} | |
| > | |
| 手書きで領収書発行 | |
| </OutlinePrimaryButton> | |
| <OutlinePrimaryButton | |
| className='min-w-52 text-base justify-center border-[#56daff] px-4 py-1.5 font-normal text-[#56daff]' |
| } catch (error) { | ||
| console.error('Failed to generate invoice PDF:', error); |
There was a problem hiding this comment.
PDFの生成に失敗した場合、console.error によるログ出力のみとなっており、ユーザーにエラーが発生したことが伝わりません。ProgressReportReceiptPdfModal.tsx のように、エラー状態を管理して画面上にメッセージを表示するか、useToast を利用して通知を行うなどのフィードバックを追加することを推奨します。
| {activity.sponsorStyles && activity.sponsorStyles.length > 0 ? ( | ||
| activity.sponsorStyles.map((link, index) => ( | ||
| <div key={`${link.sponsorStyleId}-${index}`}> | ||
| {`${link.category === 'money' ? '金' : '物'} ${link.style?.style ?? ''}${ |
There was a problem hiding this comment.
協賛スタイルのカテゴリー（'money' / 'goods'）に応じたラベルがハードコードされています。@/utils/sponsorshipActivityProgressReport で定義されている SPONSOR_STYLE_CATEGORY_LABELS を利用するか、表示用の定数を別途定義して利用することで、保守性と一貫性を高めることができます。
| {sponsorStyleLink.category === 'goods' ? ( | ||
| <MdInventory2 className='shrink-0 text-[#666666]' size={16} /> | ||
| ) : ( | ||
| <MdAttachMoney className='shrink-0 text-[#666666]' size={16} /> | ||
| )} |
There was a problem hiding this comment.
カテゴリーに応じたアイコンの切り替えロジックがハードコードされています。今後カテゴリーが増えた際の拡張性を考慮し、マッピングオブジェクトを定義するか、ユーティリティ関数に切り出すことを検討してください。
5 participants
対応Issue
resolve #0
概要
mainマージ用のプルリクエスト
画面スクリーンショット等
URLスクリーンショット
テスト項目
備考