Skip to content

Conversation

devin-ai-integration[bot]
Copy link

ProjectTechnology一覧ページにCSVからインポートする機能を実装しました。既存のcsv-import-dialog.tsxコンポーネントを使用し、アカウント一覧ページの実装と同様のパターンで実装しています。

CSVフォーマット:

  • name(必須)
  • description(任意)

重複データは上書きされます。

Link to Devin run: https://app.devin.ai/sessions/471b438c9fdf4e26875e61d7d387d575
Requested by: 村上雅彦

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

Copy link

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

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

Comment on lines 75 to 78
await client.models.ProjectTechnology.list({
filter: { name: { eq: record.name } },
limit: 1,
});
Copy link
Member

Choose a reason for hiding this comment

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

listメソッドをDynamoDBをスキャンするので効率的ではありません。
ProjectTechnology.nameにsecondaryIndexをはると、そのGSIを利用するメソッドがamplifyによって生成されるのでそのメソッドを利用してください。

Copy link
Author

Closing due to inactivity for more than 7 days.

@fossamagna fossamagna deleted the devin/1743428876-csv-import-projecttechnology branch May 7, 2025 14:56
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