Ticket #751 (closed defect: fixed)

Opened 2 years ago

Last modified 2 years ago

Email alert changes

Reported by: ed Owned by: sam
Priority: major Milestone: Maintenance
Component: Drupal modules & settings Keywords:
Cc: ed Estimated Number of Hours: 0.0
Add Hours to Ticket: 0 Billable?: yes
Total Hours: 0

Description

  1. Change the email alert template for news items to include the term 'news item' so it is:

"New news item: [title]"

  1. When users click on the subs links at the bottom of their email alerts, and they are not logged in they get the ‘access denied’ screen. This is not good. Please investigate

(a) can this be changed (with small time investment)
(b) can we change the access denied blurb to include something human encouraging the user to login to continue the journey:

"We are sorry for the inconvenience, but if you are seeing this screen having followed a link, you will probably need to login to continue with your request" (NB if they are coming from an email link with their id in it, how do we keep that journey so they get to the destinateion they wanted?)

  1. When user clicks on the general subs link at the bottom of an email alert (and is not logged in), they get a 403 forbidden page. Not good - pls investigage

Change History

comment:1 Changed 2 years ago by sam

I did some investigating of 2 & 3

The Unsubscribe URL given in the email footer is: http://www.transitionnetwork.org/notifications/unsubscribe/subscription/

When I think it should actually be:
https://www.transitionnetwork.org/notifications/unsubscribe/

It looks like the footer is set here:
https://www.transitionnetwork.org/admin/messaging/template/edit/notifications-event

I think the token used is incorrect. I have removed the [unsubscribe-url] and replaced it with [subscription-unsubscribe-url]

I think this should resolve 2. I'll await a notification and check.

Looking at 3. I spotted that logintoboggan is installed and this has the option to present the login form on access denied pages. I have enabled this and tested it. So now the user is asked to login when presented with an access denied page.

I'll take a look at 1. tomorrow.

Thanks

Sam

comment:2 Changed 2 years ago by sam

  • Status changed from new to accepted

I have now done 1)

On 2) even after changing the token it sent me to the old page. I have removed the token and put in the URL instead: https://www.transitionnetwork.org/notifications/unsubscribe/

I'll await another notification to check this works before closing the ticket..

comment:3 Changed 2 years ago by sam

  • Status changed from accepted to closed
  • Resolution set to fixed

OK the link in the emails is correct now.

If the user is logged in they go straight to managing their subscriptions.

If the user is not logged in they can enter their email to unsubscribe, or log in to manage their subscriptions.

Note: See TracTickets for help on using tickets.