Skip to content

Commit

Permalink
Merge pull request #4328 from fjordllc/bug/fix-unnecessary-publish-bu…
Browse files Browse the repository at this point in the history
…tton-appears-on-announcement-creation-page

お知らせ作成ページに不要な公開ボタンを表示しない
  • Loading branch information
komagata authored Mar 3, 2022
2 parents a222acb + 86c440c commit 1e0547f
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 1 deletion.
2 changes: 1 addition & 1 deletion app/views/announcements/_form.html.slim
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@
- if admin_or_mentor_login? && announcement.new_record?
li.form-actions__item.is-main.is-help
= f.submit '作成', class: 'a-button is-lg is-warning is-block', id: 'js-shortcut-submit'
- if admin_or_mentor_login? || announcement.published_at
- elsif admin_or_mentor_login? || announcement.published_at
li.form-actions__item.is-main.is-help
= f.submit '公開', class: 'a-button is-lg is-warning is-block', id: 'js-shortcut-submit'
li.form-actions__item.is-sub
Expand Down
6 changes: 6 additions & 0 deletions test/system/announcements_test.rb
Original file line number Diff line number Diff line change
Expand Up @@ -71,6 +71,7 @@ class AnnouncementsTest < ApplicationSystemTestCase
fill_in 'announcement[title]', with: 'タイトルtest'
fill_in 'announcement[description]', with: '内容test'

assert has_no_button? '公開'
click_button '作成'
assert_text 'お知らせを作成しました'

Expand Down Expand Up @@ -144,6 +145,7 @@ class AnnouncementsTest < ApplicationSystemTestCase
visit_with_auth '/announcements', 'kimura'
click_link 'お知らせ作成'
assert has_no_button? '作成'
assert has_no_button? '公開'
assert_text 'お知らせを作成しましたら、WIPで保存し、作成したお知らせのコメントから @mentor へ確認・公開の連絡をお願いします。'
end

Expand All @@ -153,6 +155,7 @@ class AnnouncementsTest < ApplicationSystemTestCase
within '.thread__inner' do
click_link '内容修正'
end
assert has_no_button? '作成'
assert has_button? '公開'
assert_no_text 'お知らせを作成しましたら、WIPで保存し、作成したお知らせのコメントから @mentor へ確認・公開の連絡をお願いします。'
end
Expand All @@ -163,6 +166,7 @@ class AnnouncementsTest < ApplicationSystemTestCase
within '.thread__inner' do
click_link '内容修正'
end
assert has_no_button? '作成'
assert has_no_button? '公開'
assert_text 'お知らせを作成しましたら、WIPで保存し、作成したお知らせのコメントから @mentor へ確認・公開の連絡をお願いします。'
end
Expand All @@ -173,6 +177,7 @@ class AnnouncementsTest < ApplicationSystemTestCase
within '.thread__inner' do
click_link '内容修正'
end
assert has_no_button? '作成'
assert has_button? '公開'
assert_no_text 'お知らせを作成しましたら、WIPで保存し、作成したお知らせのコメントから @mentor へ確認・公開の連絡をお願いします。'
end
Expand All @@ -183,6 +188,7 @@ class AnnouncementsTest < ApplicationSystemTestCase
within '.thread__inner' do
click_link '内容修正'
end
assert has_no_button? '作成'
assert has_button? '公開'
assert_no_text 'お知らせを作成しましたら、WIPで保存し、作成したお知らせのコメントから @mentor へ確認・公開の連絡をお願いします。'
end
Expand Down

0 comments on commit 1e0547f

Please sign in to comment.