Skip to:
Content

BuddyPress.org

Opened 9 years ago

Closed 9 years ago

#4923 closed defect (bug) (fixed)

Emails need to check for settings component

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

Description

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 9 years ago.

Download all attachments as: .zip

Change History (5)

@r-a-y
9 years ago

#1 @boonebgorges
9 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
9 years ago

  • Keywords early added

#3 @DJPaul
9 years ago

  • Keywords commit added

Looks good

#4 @boonebgorges
9 years ago

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