diff --git a/app/responders/ropensci/reviewers_due_date_responder.rb b/app/responders/ropensci/reviewers_due_date_responder.rb index 82953f8f..92a26f11 100644 --- a/app/responders/ropensci/reviewers_due_date_responder.rb +++ b/app/responders/ropensci/reviewers_due_date_responder.rb @@ -38,7 +38,7 @@ def add(reviewer) new_list = (list_of_reviewers + [reviewer]).uniq update_list("reviewers", new_list.join(", ")) update_list("due-dates", add_reviewer_due_date(reviewer).join("\n")) - respond("#{reviewer} added to the reviewers list. Review due date is #{due_date}") + respond("#{reviewer} added to the reviewers list. Review due date is #{due_date}. Thanks #{reviewer} for accepting to review! Please refer to [our reviewer guide](https://devguide.ropensci.org/reviewerguide.html).") add_collaborator(reviewer) if add_as_collaborator?(reviewer) add_assignee(reviewer) if add_as_assignee?(reviewer) process_labeling if new_list.size == 2 diff --git a/spec/responders/ropensci/reviewers_due_date_responder_spec.rb b/spec/responders/ropensci/reviewers_due_date_responder_spec.rb index 79cb959c..02fa4760 100644 --- a/spec/responders/ropensci/reviewers_due_date_responder_spec.rb +++ b/spec/responders/ropensci/reviewers_due_date_responder_spec.rb @@ -52,12 +52,12 @@ due_date = (Time.now + 10 * 86400).strftime("%Y-%m-%d") expect(due_date.strip).to_not be_empty expect(due_date).to_not eq(@new_due_date) - expect(@responder).to receive(:respond).with("@xuanxu added to the reviewers list. Review due date is #{due_date}") + expect(@responder).to receive(:respond).with("@xuanxu added to the reviewers list. Review due date is #{due_date}. Thanks @xuanxu for accepting to review! Please refer to [our reviewer guide](https://devguide.ropensci.org/reviewerguide.html).") @responder.process_message(@msg) end it "should respond to github" do - expect(@responder).to receive(:respond).with("@xuanxu added to the reviewers list. Review due date is #{@new_due_date}") + expect(@responder).to receive(:respond).with("@xuanxu added to the reviewers list. Review due date is #{@new_due_date}. Thanks @xuanxu for accepting to review! Please refer to [our reviewer guide](https://devguide.ropensci.org/reviewerguide.html).") @responder.process_message(@msg) end