send email to newly added coauthors explaining what it means

[jywarren]

This could actually be relatively simply achieved via a comment -- by automatically leaving a comment as we do for the barnstar feature:

plots2/app/controllers/tag_controller.rb

Lines 178 to 180 in dccf61b

	node.add_comment(subject: 'barnstar',
	uid: current_user.uid,
	body: "@#{current_user.username} awards a #{barnstar_info_link} to #{node.drupal_user.name} for their awesome contribution!")

The comment could say @_author_ has marked @_recipient_ as a co-author. and everybody on the thread would get email notified.

This code would go just before the last line of this block:

plots2/app/models/node.rb

Lines 660 to 667 in 033676b

	if tag.name.split(':')[0] == 'date'
	begin
	DateTime.strptime(tag.name.split(':')[1], '%m-%d-%Y').to_date.to_s(:long)
	rescue
	return [false, tag.destroy]
	end
	end
	tag.save!

and the message could look like:

"_@#{user.username} has marked @#{node.author.username} as a co-author._"

This should work but could take some troubleshooting -- happy to help, and any assistance in taking this task on is appreciated!

[grvsachdeva]

Can I take this issue ?

[jywarren]

that'd be super. Thank you!

…

On Tue, Jan 16, 2018 at 10:48 PM, Gaurav Sachdeva ***@***.***> wrote: Can I take this issue ? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#2028 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AABfJ0XEePqKabn17gWeC7U1y1yEDIL0ks5tLW2DgaJpZM4RgpfE> .

[grvsachdeva]

Hey @jywarren, the method in the tag_controller for solving this issue would work similar to method barnstar.The barnstar method is called by the form which is there in 'views/like/_like' but for making co-author there is no form present, we make co-author as for now by adding tags only, so should I add if statement in create method of tag_controller and redirect to separate method for handling creation of with:user tag or should I make a new form to trigger the method for this issue which would then perform comment creation part ?

[grvsachdeva]

Also, presently automatic comment creation for barnstar take place if the barnstar is generated via Award barnstar form and not by just adding power tag [barnstar: type], so we can also make changes so that comment creation could take place if someone has added barnstar tag.What do you think @jywarren?

[jywarren]

ah, i see. Yes, i think perhaps we should refactor these to trigger on the tag creation, not on the form submission, for both barnstars and co-authorship.

Do you want to take a crack at this? Thank you!

[jywarren]

Just watch out because you can't access current_user in the models -- they don't "know" about the user session. But we should be able to use the tag creator's uid, right?

[grvsachdeva]

I will take care of this.Thanks, Jeff.

[jywarren]

awesome!!!

…

On Tue, Jan 23, 2018 at 12:16 PM, Gaurav Sachdeva ***@***.***> wrote: I will take care of this.Thanks, Jeff. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#2028 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AABfJxe7RTD3PIZWOZ58kjT8jkR7sM74ks5tNhPygaJpZM4RgpfE> .

[grvsachdeva]

The comment could say @author has marked @recipient as a co-author. and everybody on the thread would get email notified.

I want to confirm @jywarren that whether we need to send email to all the people present on that thread or only to the user who has been added as a co-author ?

[jywarren]

Only to the co-author -- good question! Thanks!

…

On Tue, Feb 6, 2018 at 1:12 PM, Gaurav Sachdeva ***@***.***> wrote: The comment could say @*author* has marked @*recipient* as a co-author. and everybody on the thread would get email notified. I want to confirm @jywarren <https://github.com/jywarren> that whether we need to send email to all the people present on that thread or only to the user who has been added as a co-author ? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#2028 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AABfJ4kS_6dgpQbMR0nj1wrqZxKNKlQAks5tSJYrgaJpZM4RgpfE> .

[grvsachdeva]

Thanks for clarifying Jeff