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.

Sign up for GitHub

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

Jump to bottom

GET /api/v3/stampsのレスポンスにhasThumbnailを含める #1154

Closed
motoki317 opened this issue Apr 23, 2021 · 4 comments · Fixed by #1929
Closed

GET /api/v3/stampsのレスポンスにhasThumbnailを含める #1154

motoki317 opened this issue Apr 23, 2021 · 4 comments · Fixed by #1929
Assignees
@azbcww
Labels
good first issue 比較的取り組みが簡単

Comments

@motoki317
Copy link
Member

サムネイル画像を代わりに表示するため

(レスポンスが)正規化されないのでちょっと気持ち悪い
@motoki317 motoki317 added the good first issue 比較的取り組みが簡単 label Jul 1, 2021
@azbcww azbcww self-assigned this Aug 9, 2023
@azbcww
Copy link
Contributor

azbcww commented Aug 11, 2023

これってv7file構造体のようにHasThumbailを追加したStamp構造体に修正するのか，レスポンスの直前のデータにhasThumbnailを追加するのか，どちらを想定していますか？

@logica0419
Copy link
Member

サムネイルの情報を取得するのが、コンピューター視点でどれくらい大変かによるかなぁと思います

(前提として、言ってくれたStamp構造体はDBの構造(スキーマ)を定義している構造体です)

基本的にはわざわざDBに入れると更新の手間が増えてしまうのでその場で取得という形にしてほしいです
ただ、もしサムネイルを持っているかどうかの判定がめちゃくちゃ重くてまともに使えないというレベルであれば、DBに入れておくのはアリな戦略です(多分今回はそんなことないと思いますが)

とりあえず、サムネイルがあるかどうか判定するロジックがすでにtraQにあるはずなので、それを探すところからやってみて下さい〜

@azbcww
Copy link
Contributor

azbcww commented Aug 12, 2023

ありがとうございます！了解です！
とりあえずロジック探してみますー

@azbcww azbcww linked a pull request Aug 14, 2023 that will close this issue
レスポンスにサムネイル情報追加 #1929
Merged
@azbcww
Copy link
Contributor

azbcww commented Aug 14, 2023

draft作成したので時間に余裕あるとき見てもらいたいです

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@azbcww azbcww

Labels
good first issue 比較的取り組みが簡単
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

レスポンスにサムネイル情報追加 traPtitech/traQ
3 participants
@motoki317 @azbcww @logica0419