Sends a DM to the user on thread create

[EthanThatOneKid]

Unexpected behavior

The Sendlimiter bot sends DMs to the individual if their message is deleted. However, Sendlimiter tries to send DMs for some message that are not deleted.

Desired behavior

Only send DMs of a deleted message.

Strategy

Attempt to execute the message delete operation before attempting to send a DM.

Relevant code location

sendlimiter/main.go

Lines 51 to 73 in 0d28108

	// Send a DM to the user.
	channel, err := s.CreatePrivateChannel(m.Author.ID)
	if err != nil {
	log.Println("Failed to create private channel:", err)
	return
	}
	if _, err := s.SendMessageComplex(channel.ID, api.SendMessageData{
	Content: "Please attach an image or a link to a showcase channel.",
	}); err != nil {
	log.Println("Failed to send message:", err)
	}
	if _, err := s.SendMessageComplex(channel.ID, api.SendMessageData{
	Content: m.Content,
	}); err != nil {
	log.Println("Failed to send message:", err)
	}
	// Delete the message.
	if err := s.DeleteMessage(m.ChannelID, m.ID, "No attachments"); err != nil {
	log.Println("Failed to delete message:", err)
	}

[EthanThatOneKid]

Suggested PR title: Attempt message delete before DM.

[EthanThatOneKid]

The suggested strategy was implemented as part of #8, but unfortunately did not work as expected and is now deleting messages without our consent.

cc @karnikaavelumani

[EthanThatOneKid]

The suggested strategy was implemented as part of #8, but unfortunately did not work as expected and is now deleting messages without our consent.

cc @karnikaavelumani

As a result, we've rolled back the merge in ef74f8e.