#210 Post モデルに similarities プロパティ追加

Closed
opened 1 day ago by みてるぞ · 1 comments

post_similarities_as_postpost_similarities_as_target_post を統合して similarities として扱ひたい.
related メソッドもついでに直す.

`post_similarities_as_post` と `post_similarities_as_target_post` を統合して `similarities` として扱ひたい. `related` メソッドもついでに直す.
みてるぞ added the
area/backend
label 1 day ago
みてるぞ added the
type/bug
label 1 day ago
みてるぞ added the
P2
label 1 day ago
みてるぞ added the
status/ready
label 1 day ago
みてるぞ removed the
type/bug
label 1 day ago
みてるぞ added the
type/task
label 1 day ago
みてるぞ commented 1 day ago
Owner

target_post をキーにした場合は postpost をキーにした場合は target_post が対象となり,一致しない.
そのため,統合したところで場合分けは必要となり,あまり恩恵が得られないと考へる.

やるのであれば related メソッドの ids_with_cos だけ切り出すとかが意味ある変更になるが,必要になったらその都度やればいいだけの話.

よって,却下.

`target_post` をキーにした場合は `post`,`post` をキーにした場合は `target_post` が対象となり,一致しない. そのため,統合したところで場合分けは必要となり,あまり恩恵が得られないと考へる. やるのであれば `related` メソッドの ids_with_cos だけ切り出すとかが意味ある変更になるが,必要になったらその都度やればいいだけの話. よって,却下.
みてるぞ closed this issue 1 day ago
みてるぞ removed the
P2
label 1 day ago
みてるぞ removed the
status/ready
label 1 day ago
みてるぞ added the
wontfix
label 1 day ago
Sign in to join this conversation.
No Milestone
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.