Skip to content

Conversation

@irumaru
Copy link
Member

@irumaru irumaru commented Mar 3, 2025

注意: 開発中のメモを含むため、レビュー段階前にPRを作りました。そのため、まだマージしないでね!

@irumaru
Copy link
Member Author

irumaru commented Mar 3, 2025

db,api,configできた
次、interfaceに関数を実装する

@irumaru
Copy link
Member Author

irumaru commented Mar 3, 2025

メモ
golangの仕様では、各モジュール初期化時にinit関数が最初に実行されます

@irumaru
Copy link
Member Author

irumaru commented Mar 7, 2025

Controllerを一部実装した

@irumaru
Copy link
Member Author

irumaru commented Mar 10, 2025

VideoLog以外のAPIを実装

@irumaru
Copy link
Member Author

irumaru commented Mar 10, 2025

レビューしてほしい場所

  • データベース構造: api/db/schemas.go
  • OASの若干の変更: oapi/api.yml

このプルリクエストより後に行う予定の変更

  • user_idの追加
  • 認証
  • SeriesTagMapテーブルの追加
  • OpenAPIの関係ないフィールド(Post時のIDなど)を隠す

@irumaru irumaru requested a review from pantyetta March 10, 2025 14:10
Copy link
Collaborator

@pantyetta pantyetta left a comment

Choose a reason for hiding this comment

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

/anime/season/{id}
/anime/episode/{id}
のGETが見当たらない。。。多分これから実装かな?

Title string
Synopsis string
Cours string
Cast string
Copy link
Collaborator

Choose a reason for hiding this comment

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

castって複数いないっすか?

Copy link
Member Author

Choose a reason for hiding this comment

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

/anime/season/{id}
/anime/episode/{id}
これから実装ですね

castは直します!

@irumaru
Copy link
Member Author

irumaru commented Mar 10, 2025

直しました

Synopsis string
Cours string
Cast string
Casts string
Copy link
Collaborator

Choose a reason for hiding this comment

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

あ、伝わってなかった、、、すいません!
castsってstringで良いのか?tagみたいに別で持たせないといけないのでは?って事でした

Copy link
Member Author

Choose a reason for hiding this comment

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

casts、ただ表示するだけを想定するなら、stringで"AA、BB、CC"って入力すればよいと思ってた
例えば、AAをクリックすると、AAキャストのページに飛んだり、AAが出演している他のコンテンツを表示したりするなら、RDBの機能を使いたいので、tagと同じように別で持たせた方がよさそう

どちらを想定していますか?

Copy link
Collaborator

Choose a reason for hiding this comment

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

後者のAAクリックするとAAキャストページに飛ぶのを想定してました

Copy link
Member Author

Choose a reason for hiding this comment

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

アニメ録画に特化させるので、その方が良さそう!
OASの大きな変更が必要なので、わかりやすさのためにPR分けます

@irumaru irumaru force-pushed the sharksucker-add-api-server branch from e87d0fb to b5f5cff Compare March 11, 2025 09:09
@irumaru irumaru requested a review from ryomanden as a code owner March 18, 2025 15:52
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.

3 participants