-
Notifications
You must be signed in to change notification settings - Fork 72
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
ブログ記事公開時に通知先のターゲットを選択できるようにした #8346
Draft
ayu-0505
wants to merge
18
commits into
main
Choose a base branch
from
feature/set-article-notification-target
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
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
ecff593
to
69b282e
Compare
69b282e
to
b3cbcaa
Compare
@ayu-0505 連絡ありがとうございます!デザイン了解ですー |
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.
Issue
概要
ブログ記事を公開する際に通知先を選択できるようにターゲットをラジオボタンで選択できるように変更しました。
選択肢は「全員(退会者を除く)」、「現役生のみ」、「就職希望者のみ」、「通知しない」の4つで、「通知しない」を選択した場合はメンター・管理者含めどこにも通知が飛ばないようになっています。
通知は初回公開時のみに行われるため、「公開するボタン」をクリックすると、「通知を行う旨と通知先」をお知らせするダイアログが表示されます。
通知送付後、一度公開した記事の編集画面では通知をすでに行ったというメッセージが表示され、ターゲット選択のラジオボタンは非表示となっています。
変更確認方法
git fetch origin pull/8346/head:feature/set-article-notification-target
g switch feature/set-article-notification-target
foreman start -f Procfile.dev
でサーバーを起動するkomagata
でログインする。(pass:testtest
)編集画面の確認
この記事を公開すると、全員(退会者を除く)に通知が送られます。もし記事に誤った情報が含まれている場合、その通知を受け取った人を介してSNSなどで拡散される可能性があります。そのため、記事内の年月日などに間違いがないか、しっかり確認してください。公開しても問題ありませんか?
)全員(退会者を除く)
、現役生のみ
、就職希望者のみ
)通知しない
の場合の警告ダイアログは「通知しない」が選択されています。よろしいですか?
となっているか確認する。この記事は初回公開時にxxxxのみに対してすでに通知を行っています。
といった通知をすでに行った旨のメッセージが表示され、選択ボタンは表示されなくなっているかを確認する。(通知先は先ほど選んだ通知先となっています。)
この記事は通知を行わずに公開しました。
というメッセージとなっている。通知先の確認
全員
、現役生
、就職希望者
、通知なし
) 本文部分は適当に入力する。komagata
のままトップページに飛び、記事の作成者には通知が飛んでいないことを確認する。testtest
)、通知の有無を確認する。(「通知なし」の記事はいずれのユーザーにも飛ばない)Screenshot
変更前
変更後