Thanks heaps for getting the notification working for FAQ_Ask.

Is it possible for a link to be added to the email that gets sent to the Expert that they can click on to take them directly to the Answer page? Just trying to make it as easy and simple for Experts to get the job done with the minimum amount of fuss and bother to them.

Again, thank you very much for the work you've put into this FAQ addition.

CommentFileSizeAuthor
#12 fixed-i-think.jpg19.28 KBrastarr
#7 Question_waiting.jpg28.35 KBrastarr

Comments

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented

Hmm. I'd have to move a little of the code around, but this shouldn't be that difficult. I'll try to get to it today.

I'm not sure how much good it will do, though, because my site requires the expert to log in before they can answer. That's why the block is there.

rastarr’s picture

rastarr commented

Yes I agree the login first is an issue though the simple answer might be:

Preface the email link with some short instructions like:

=================
You've received a question to answer.

[1] Firstly you'll probably need to log into the website and you can do that by clicking here:
[2] Now you've logged in, you can quickly get to the edit area to answer the question by clicking here:
=================

Just a thought that may help the workflow a little.

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented
Assigned: Unassigned » nancydru

Good idea. Thanks.

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented
Title: new subscriptions addition » Notifcation email improvements
Status: Active » Fixed

I've committed a fix.

In order to include those links, I had to force HTML format emails.

rastarr’s picture

rastarr commented

Cool - just downloaded the dev and got an email notification - I gather the links and such are yet to be included cause the dev version I installed didn't provided that yet.
All good though

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented

Hmm, I just looked at CVS and the new code is there as of 17:59 (Drupal time) on Nov. 28 - the -dev size is 14.2kb.

rastarr’s picture

rastarr commented
StatusFileSize
new Question_waiting.jpg28.35 KB

Just redownloaded it and there a few hundred bytes difference and upon installing, it includes links in the email - mind you, the links don't actually work but it's getting there :)

I've attached a screenshot of the layout and email. Nothing in the email body is clickable as a proper formed URL might be. It's like the proceeding 'a' part of the tag has been swallowed up and mangles the email's HTML.

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented
Status: Fixed » Postponed (maintainer needs more info)

Hmm, it worked just fine for me with MS Outlook. What email program are you using? Does it handle other HTML mail correctly?

rastarr’s picture

rastarr commented

I use Mail under OSX - it doesn't have any issue with HTML for anything else.

I also checked it using WebMail and it comes out the same way. The site is using Mime Mail v5.x-1.x-dev for production of HTML - is this what you are using, Nancy?

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented

No, I create my own mail headers - maybe there's something wrong with them. Does your system give any hints? Like I said, it works fine here.

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented

Here's a possibility: go into the module to line 248, which is:

    $body = wordwrap($body);

And just comment it out (add '//' in the first two columns).

I just looked at your picture again and I'm wondering if this line could be the problem.

rastarr’s picture

rastarr commented
StatusFileSize
new fixed-i-think.jpg19.28 KB

Cool Nancy - that seems to have solved it

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented
Status: Postponed (maintainer needs more info) » Needs review

Great, I think. What a weird thing to break it. Okay, I'll remove that line.

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented
Status: Needs review » Fixed

Committed to -beta3

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)

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

verta’s picture

verta commented
Title: Notifcation email improvements » Notification email improvements

typo, for search