Skip to:

Opened 11 years ago

Closed 11 years ago

#4923 closed defect (bug) (fixed)

Emails need to check for settings component

Reported by: r-a-y's profile r-a-y Owned by:
Milestone: 1.7.1 Priority: normal
Severity: normal Version: 1.7
Component: Settings Keywords: has-patch early commit


Since the settings component is not one of the autoloaded components in BP 1.7, we need to remove the "disable notifications" line from all BP emails when the settings component is diabled.

Otherwise, when a user clicks on the /settings/notifications/ link, the user will get a 404.

Attachments (1)

4923.01.patch (7.1 KB) - added by r-a-y 11 years ago.

Download all attachments as: .zip

Change History (5)

11 years ago

#1 @boonebgorges
11 years ago

  • Component changed from Core to Settings
  • Milestone changed from 1.7 to 1.7.1

Good catch. Not really a regression, but definitely more visible now that Settings isn't active by default.

#2 @johnjamesjacoby
11 years ago

  • Keywords early added

#3 @DJPaul
11 years ago

  • Keywords commit added

Looks good

#4 @boonebgorges
11 years ago

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.