Infrastructure/ghost
0
Fork 0
mirror of https://github.com/TryGhost/Ghost.git synced 2025-02-03 23:00:14 -05:00
Code Issues Activity

Updated default newsletter subscriptions for member

Browse source 
refs https://github.com/TryGhost/Team/issues/1469

Previously, members were subscribed to all available newsletters by default when added. This change updates the default newsletters subscription for member to take into account newsletter preferences for auto opt-in as well as visibility.
This commit is contained in:
Rishabh 2022-04-14 21:10:14 +05:30
parent 415698f0f4
commit f9ce2d9064
1 changed files with 11 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
ghost/members-api/lib/repositories/member.js
View file

@ -197,9 +197,15 @@ module.exports = class MemberRepository {
memberStatusData.status = 'comped'; memberStatusData.status = 'comped';
} }
// Subscribe member to all newsletters by default // Subscribe member to default newsletters
if (!memberData.newsletters && this._labsService.isSet('multipleNewsletters')) { if (!memberData.newsletters && this._labsService.isSet('multipleNewsletters')) {
const newsletters = await this._newslettersService.browse(_.pick(options, 'transacting')); const browseOptions = _.pick(options, 'transacting');
// By default subscribe to all active auto opt-in newsletters with members visibility
//TODO: Will mostly need to be updated later for paid-only newsletters
browseOptions.filter = 'status:active+subscribe_on_signup:true+visibility:members';
const newsletters = await this._newslettersService.browse(browseOptions);
memberData.newsletters = newsletters || []; memberData.newsletters = newsletters || [];
} }