Skip to content

Commit 8133f91

Browse files
committed
remove extra way to attach to discussion
fixes CNVS-29918 Test Plan: - In a course, navigate to the Settings page - On the Course Details tab, click on the "more options" link at the bottom of the page - Make sure that the "Let students attach files to discussions" option is checked - Navigate to a discussion in the course - Begin creating a new reply - Note that there is an "Attach" option in the bottom left - Post the reply - Click on the Cog menu on the post and choose "Edit" - Note that the Attach option is not there Change-Id: Ib05e14107adeb934b51e60a04f73cc083d5fc520 Reviewed-on: https://gerrit.instructure.com/95678 Tested-by: Jenkins Reviewed-by: Clay Diffrient <cdiffrient@instructure.com> QA-Review: David Tan <dtan@instructure.com> Product-Review: Steven Burnett <sburnett@instructure.com>
1 parent 94da57b commit 8133f91

2 files changed

Lines changed: 3 additions & 10 deletions

File tree

app/views/jst/discussions/_reply_form.handlebars

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -29,14 +29,6 @@
2929
id="{{#if root}}root_{{/if}}reply_message_for_{{id}}"></textarea>
3030

3131
<div class="clearfix">
32-
<div class="discussion-reply-attachments-container">
33-
<ul class="discussion-reply-attachments"></ul>
34-
{{#if canAttach}}
35-
<a href="#" class="discussion-reply-add-attachment discussion-reply-action icon-paperclip"
36-
data-event="addReplyAttachment">{{#t "attach_file"}}Attach{{/t}}</a>
37-
{{/if}}
38-
</div>
39-
4032
<div class="discussion-reply-buttons">
4133
<a href="javascript:" class="cancel_button btn btn-small">{{#t "cancel"}}Cancel{{/t}}</a>
4234
<button class="btn btn-primary btn-small" type="submit">{{#t "post_reply"}}Post Reply{{/t}}</button>

spec/selenium/helpers/discussions_common.rb

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -64,9 +64,10 @@ def add_reply(message = 'message!', attachment = nil)
6464

6565
if attachment.present?
6666
filename, fullpath, data = get_file(attachment)
67-
@last_entry.find_element(:css, '.discussion-reply-add-attachment').click
67+
f('#ui-id-2').click
6868
wait_for_ajaximations
69-
@last_entry.find_element(:css, '.discussion-reply-attachments input').send_keys(fullpath)
69+
f('#editor_tabs_3 > div:nth-child(3) > a').click
70+
f('#attachment_uploaded_data').send_keys(fullpath)
7071
end
7172

7273
submit_form(@last_entry.find_element(:css, ".discussion-reply-form"))

0 commit comments

Comments
 (0)