There are two strings to be changed:
In Message can't be sent to @name because he has been blocked the 'he' must be changed by 'it'.

In More actions.. there should be another . at the end.

For translation purposes: could you tell me what '!undo' can be in The previous action can be !undo.. For translation purposes it is mostly better to have separate strings but not variables.

Comments

naheemsays’s picture

Another string that can potentially be improved from the email notifications module:

Hi !username,\n\nThis is an automatic reminder from the site !site. You have received a new private message from !author.\n\nTo read your message, follow this link:\n!uri/messages\n\nIf you don't want to receive these emails again, change your preferences here:\n!uri/messages/notify

EDIT - as a reply to the last post, currently !undo is a place holder for the word undone. Not the best IMO and I would prefer if the text was somehow changed to reflect what actually happens. "You have marked the conversations as read. Mark as unread" would IMO be better but it may take adding more logic.

berdir’s picture

StatusFileSize
new1.56 KB

!undo is work in progress, I'm not happy about it either :) The idea is to make it configurable somehow.

Attached is a patch to fix the two mentioned strings. Note that the first is is only an example and is not actually used.

berdir’s picture

Status: Active » Needs review
berdir’s picture

StatusFileSize
new2.49 KB

Re-rolled to use the same string as we do in pm_block_user.module. I also changed ! to %, as the username needs to be escaped afaik.

litwol’s picture

Status: Needs review » Fixed

Committed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.