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

KeepMemo #444

Merged
merged 14 commits into from
Dec 2, 2024
Merged

KeepMemo #444

merged 14 commits into from
Dec 2, 2024

Conversation

RRRyoma
Copy link
Contributor

@RRRyoma RRRyoma commented Oct 8, 2024

PRの概要

オートマッチの機能を作った。後々には運営アカウントのチャットを作ったり、問い合わせ機能の移管などができる。

closes #418

具体的な変更内容

影響範囲

動作要件

DBにID=0の"Keepメモ"を作ってください。
いらすとやの
https://blogger.googleusercontent.com/img/b/R29vZ2xl/AVvXsEhvpbYflYCwbG_c11ADWhZUaf93zrtmvYYjSvY4NNxcF4Ri-XO6jiFZq-1InXfcxBjTD9_8jQntvnzML5F0geA04H9etzy3dcZ7SaqpbfKX4PmFgg8nplhaSLBCWo6zOIwq-jJc9tjrXxKV/s1600/bunbougu_memo.png
をpictureUrlに用いるのがおすすめ

補足

レビューリクエストを出す前にチェック!

  • 改めてセルフレビューしたか
  • 手動での動作検証を行ったか
  • server の機能追加ならば、テストを書いたか
    • 理由: 書いた | server の機能追加ではない
  • 間違った使い方が存在するならば、それのドキュメントをコメントで書いたか
    • 理由: 書いた | 間違った使い方は存在しない
  • わかりやすいPRになっているか

@RRRyoma
Copy link
Contributor Author

RRRyoma commented Oct 8, 2024

CourseMate.-.Google.Chrome.2024-10-08.20-38-28.-.Compressed.with.FlexClip.mp4

Copy link
Contributor

@KaichiManabe KaichiManabe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

メモを導入する上で、guidとか必要になる気がするんですが、
メモのDBの構造を教えてください!

@RRRyoma
Copy link
Contributor Author

RRRyoma commented Oct 9, 2024

name,pictureUrl,idは送った通りですが、他は適当で大丈夫です!

@aster-void
Copy link
Contributor

サーバー側でユーザー作成時に登録するのはだめなんですか?

Copy link
Contributor

@KaichiManabe KaichiManabe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

僕が巻き取りますね!

Copy link
Contributor

@KaichiManabe KaichiManabe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

事前に、データベースにメモのアカウントを作っておいたら、メモと自動マッチングするようにしました。
アカウントを作ったら自動でサーバーからマッチングを飛ばす感じになっています。
その気になれば、運営アカウントとマッチングみたいなのはすぐに可能です
#535 をMergeした後にMergeしたいっす

@KaichiManabe KaichiManabe self-requested a review December 2, 2024 09:03
@KaichiManabe KaichiManabe merged commit 84a13ab into main Dec 2, 2024
8 checks passed
@KaichiManabe KaichiManabe deleted the memo branch December 2, 2024 09:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

チャット画面で、メモを可能にする
3 participants