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

リアクションデッキAPIのリファクタリング #838

Open
kmycode opened this issue Sep 6, 2024 · 0 comments
Open

リアクションデッキAPIのリファクタリング #838

kmycode opened this issue Sep 6, 2024 · 0 comments

Comments

@kmycode
Copy link
Owner

kmycode commented Sep 6, 2024

挙動を変更してほしい機能や動作

https://github.com/kmycode/mastodon/blob/9e1c63aa2c48406fe5d445a530b4d36127dfd425/app/controllers/api/v1/reaction_deck_controller.rb

現在の挙動

プログラム内でハッシュを組み立ててそれをJSONに変換する

変更してほしい新しい挙動

モデルクラスの配列を作って、それをSerializerを使ってJSONに変換する

必要性

他のコードのやり方に合わせる
JSONを手動で組み立てているのはここくらいだし、メタ情報の消し方を見てもセキュリティ上の潜在的な危険性をうかがわせる

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant