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

`ProcessReferenceService`のリファクタリング #909

Open

kmycode opened this issue Nov 13, 2024 · 0 comments

Labels

Owner

kmycode commented Nov 13, 2024

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

ProcessReferenceService（投稿から参照URLを抜き出し、投稿に参照情報を追加する）

現在の挙動

ローカルでまだ取得していない投稿を参照として追加する時、新たに自身を呼び出す非同期Workerを作成する
クラスメソッドの呼び出しにより、同期・非同期処理を選択する

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

投稿IDとURIをパラメータにして投稿に参照を追加するだけのWorkerを新たに作成する
Sidekiq::Workerのperformとperform_asyncを使い分ける方法に変更する

必要性

冪等性の確保、責務を分けることによるコードのシンプル化
他とやり方を揃えて保守性を上げる

The text was updated successfully, but these errors were encountered:

kmycode added the refactoring label

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