グループ外NGのときorグループ外NGに変更するときにwarningを出す #922
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ras0q
requested changes
Jan 5, 2024
Comment on lines
36
to
37
| グループ外の人が参加できない場合，グループ外の人にはイベントの作成通知が届きません。<br /> | ||
| また，途中でこの設定を変更した場合，すでに参加登録していたグループ外の人にも通知が届かなくなります。 |
There was a problem hiding this comment.
通知はtraQで各自いじれるので触れてほしいのは通知関係の話よりは既に登録されていたメンバーが削除される可能性があるってことなのでこれに言及したwarningにしてほしいです 🙏
#893 (comment)
There was a problem hiding this comment.
承知いたしました。確認ありがとうございます:pray:
itt828
reviewed
Jan 5, 2024
| type="warning" | ||
| elevation="1" | ||
| class="mt-3" | ||
| max-width="900px" |
There was a problem hiding this comment.
他のコンポーネントにもついてないからmax-widthなくてもよさそう
There was a problem hiding this comment.
確かこれがないと見た目が悪くなってしまっていたのでつけた記憶があります。
ただあまりスマートでないと思っていたので，もう少し別の方法を探してみます:pray:
もしかしたら，また相談させていただくかもしれません。
ras0q
reviewed
Jan 6, 2024
| elevation="1" | ||
| class="mt-3" | ||
| > | ||
| この設定にすると，グループ外の参加者の情報が存在すれば全て削除されます。 |
There was a problem hiding this comment.
イベント作成時は誰の参加情報も登録されてなくて削除することがないので可能なら更新時だけ出したいですね
更新時はグループ外の参加者を許可しない場合、既に登録されているグループ外の参加者の参加情報は削除されますとか？
|
よさそう |
itt828
approved these changes
Jan 10, 2024
|
@ras0q approveお願いします:pray: |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
issue893です。
確認よろしくお願いいたします。