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

Memory作成時のCategory登録について検討する #312

Open
2525nicole opened this issue Oct 31, 2024 · 0 comments
Open

Memory作成時のCategory登録について検討する #312

2525nicole opened this issue Oct 31, 2024 · 0 comments

Comments

@2525nicole
Copy link
Owner

2525nicole commented Oct 31, 2024 •
edited
Loading

現状の実装について以下のような問題点を感じています。

  • MemoriesControllerコントローラの実装が肥大化している
  • MemoryモデルでCategoryに関する処理をしている
  • Memoryモデルでコントローラのparamsを参照している
    これらの問題を解決できないか検討いただきたいです。具体的には次のような手があると考えています。
  • モデルのコールバック( before_save など)を使う
  • accepts_nested_attributes_for を使って、親子を同時に保存する
    自分もやったことがないので本当に機能するかはわからないのですが、子から親に対しても指定ができるようです
    https://donghai821.hatenadiary.org/entry/20110804/1312475967
@2525nicole 2525nicole converted this from a draft issue Oct 31, 2024
@2525nicole 2525nicole mentioned this issue Nov 4, 2024
Merged
@2525nicole 2525nicole moved this from 作業中 to 開発MTGで報告 in memo-recall Nov 6, 2024
@2525nicole 2525nicole moved this from 開発MTGで報告 to やること in memo-recall Nov 6, 2024
@2525nicole 2525nicole moved this from やること to 開発MTGで報告 in memo-recall Nov 6, 2024
@2525nicole 2525nicole mentioned this issue Nov 12, 2024
Closed
@2525nicole 2525nicole moved this from 開発MTGで報告 to 完了 in memo-recall Nov 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
memo-recall
Status: 完了
Milestone
No milestone
Development

No branches or pull requests
1 participant
@2525nicole