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

csvで協賛企業のアップロードを行うAPIの作成&返り値取得のためrowAffeted件数分の取得を作成 #881

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

hikahana
Copy link
Collaborator

@hikahana hikahana commented Oct 9, 2024

対応Issue

resolve #878

概要

  • csvでの複数挿入APIを作成しました。

  • usecaseに関数をいくつか作成しました。csv内にまとめた方がよければ共有ください。

  • バリデーションとしてcsvの中身が空か、ヘッダーが正しいか、空のレコードが含まれているかを実装しました。

  • 単体or複数の挿入に対応させるためrowAffectedで取得した件数分getを行うAPIも作成しました。

  • returingがあればいらなさそうですが今のバージョンだと非対応でした。

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

  • URL
    スクリーンショット
    image
    image

テスト項目

  • http://localhost:1323/swagger/index.html#/sponsor/post_sponsors_csvにアクセスしてcsvファイルを挿入して実行してください
  • 備考欄に私が使用したcsvファイルを共有しています。
  • 挿入が成功していればokです。
  • ロジック周りのコードレビューもお願いします。主にussecase

備考

単体挿入
test_csv.csv

複数挿入
test_csvs.csv

ヘッダーミス
dummy_csvs.csv

テストコード書いてみたい感

@hikahana hikahana requested a review from Kubosaka October 9, 2024 12:15
@hikahana hikahana self-assigned this Oct 9, 2024
Copy link
Collaborator

@Kubosaka Kubosaka left a comment

Choose a reason for hiding this comment

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

動作おけです

api/internals/usecase/sponsor_usecase.go Show resolved Hide resolved
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.

協賛企業のcsv一括登録APIの作成
2 participants