Skip to content

ProjectTechnologyの詳細ページを追加#31

Merged
fossamagna merged 5 commits intomainfrom
devin/1743146544-add-project-technology-detail-page
Mar 28, 2025
Merged

ProjectTechnologyの詳細ページを追加#31
fossamagna merged 5 commits intomainfrom
devin/1743146544-add-project-technology-detail-page

Conversation

@devin-ai-integration
Copy link

@devin-ai-integration devin-ai-integration bot commented Mar 28, 2025

ProjectTechnologyの詳細ページを追加

パス: /project-technologies/:projectTechnologyId

実装内容

  • ProjectTechnologyの基本情報(名前、説明)の表示
  • 関連プロジェクトのリスト表示(shadcn/uiのtableコンポーネントを使用)
    • プロジェクト名をクリックすると詳細ページに遷移
  • 編集ページへのリンク
  • 削除機能(AlertDialogを使用した確認ダイアログを独立したコンポーネントとして実装)

UIデザインは既存のページと同様のスタイルで実装しています。

Link to Devin run: https://app.devin.ai/sessions/acb2348e1aa2403eadb1f5eb1ab208d8
Requested by: 村上雅彦 (m-murakami@esm.co.jp)

Co-Authored-By: 村上雅彦 <m-murakami@esm.co.jp>
@devin-ai-integration
Copy link
Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add "(aside)" to your comment to have me ignore it.
  • Look at CI failures and help fix them

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

Co-Authored-By: 村上雅彦 <m-murakami@esm.co.jp>
@aws-amplify-ap-northeast-1
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-31.d3ojkf5oa8o7rk.amplifyapp.com

Co-Authored-By: 村上雅彦 <m-murakami@esm.co.jp>
Comment on lines +12 to +19
<div>
<Button variant="destructive" type="submit" form="delete-form">
Delete
</Button>
<form id="delete-form" method="post" className="hidden">
<input type="hidden" name="action" value="delete" />
</form>
</div>
Copy link
Member

Choose a reason for hiding this comment

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

以下のように、ProjectTechnologyDeleteDialogを作成する前に使っていたダイアログを表示するようにしてください。

Suggested change
<div>
<Button variant="destructive" type="submit" form="delete-form">
Delete
</Button>
<form id="delete-form" method="post" className="hidden">
<input type="hidden" name="action" value="delete" />
</form>
</div>
<AlertDialog>
<AlertDialogTrigger asChild>
<Button variant="destructive">Delete</Button>
</AlertDialogTrigger>
<AlertDialogContent>
<AlertDialogHeader>
<AlertDialogTitle>Are you sure?</AlertDialogTitle>
<AlertDialogDescription>
This action cannot be undone. This will permanently delete the
technology and remove it from all associated projects.
</AlertDialogDescription>
</AlertDialogHeader>
<AlertDialogFooter>
<AlertDialogCancel>Cancel</AlertDialogCancel>
<form method="post">
<input type="hidden" name="action" value="delete" />
<AlertDialogAction
type="submit"
className="bg-destructive text-destructive-foreground hover:bg-destructive/90"
>
Delete
</AlertDialogAction>
</form>
</AlertDialogFooter>
</AlertDialogContent>
</AlertDialog>

devin-ai-integration bot and others added 2 commits March 28, 2025 07:49
Co-Authored-By: 村上雅彦 <m-murakami@esm.co.jp>
@fossamagna fossamagna merged commit f251f86 into main Mar 28, 2025
2 checks passed
@fossamagna fossamagna deleted the devin/1743146544-add-project-technology-detail-page branch May 7, 2025 14:55
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.

1 participant