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

リマインド機能の改善 #93

Merged
merged 5 commits into from
May 1, 2022
Merged

Conversation

na2na-p
Copy link
Contributor

@na2na-p na2na-p commented Apr 14, 2022

これは何

タイトルの通り。
大きく3つの変更点があります。

renote対象が消されていた場合にリマインダー解除を行う機能

#89 に関係しそう。
「やった」「やめる」でリマインダー解除する場所と同じように書きました。

何が変わるの

リマインダーで、todoしてるノートが消されたら自動的にリマインドも解除する。

リマインド対象のノートのvisibilityがfollowersのときにtodoを受け付けないようにした

todoをお願いしたノートのvisibilityが、followersとなっていた時に🆗とはなる。が、引用RNができなくてただ覚えているだけになっているので、それだったら最初から受け付けないようにすればいいんじゃない?となり書きました。

何が変わるの

  • msg.visibilityで公開範囲が取れるようになる
  • リマインダーで、todoしてるノートのvisibilityが、followersだったら受け付けなくなる
  • 受け付けられなかったときにNGのリアクションを返すようになる

#70 他人のリマインダーを操作できるバグの修正

「やった」発言をしたユーザーidと、リマインド登録したユーザーidを比較し、一致しなければ画像の一番下のように返すように変更し、リマインダー解除しないようにした
画像

@syuilo syuilo merged commit 5cd9e17 into syuilo:master May 1, 2022
@na2na-p
Copy link
Contributor Author

na2na-p commented May 1, 2022

先の箇所の分岐条件を、400だけチェックに変更しました。

@syuilo
Copy link
Owner

syuilo commented May 1, 2022

🙏🙏🙏

@na2na-p na2na-p deleted the reminderFix branch May 1, 2022 14:39
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