Skip to content

Commit 75da926

Browse files
authored
fixed member import email setting import (#16269)
refs TryGhost/Product#2605 -updated unparse to look at both subscribed and subscribed_to_emails -subscribed is for backwards compatibility -may want to retire subscribed since we can't set from front-end
1 parent dd35989 commit 75da926

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

packages/members-csv/lib/unparse.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ const unparse = (members, columns = DEFAULT_COLUMNS.slice()) => {
4848
email: member.email,
4949
name: member.name,
5050
note: member.note,
51-
subscribed_to_emails: member.subscribed,
51+
subscribed_to_emails: member.subscribed || member.subscribed_to_emails ? true : false,
5252
complimentary_plan: member.comped || member.complimentary_plan,
5353
stripe_customer_id: _.get(member, 'subscriptions[0].customer.id') || member.stripe_customer_id,
5454
created_at: member.created_at,

packages/members-csv/test/unparse.test.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,14 +13,14 @@ describe('unparse', function () {
1313

1414
assert.ok(result);
1515

16-
const expected = `id,email,name,note,subscribed_to_emails,complimentary_plan,stripe_customer_id,created_at,deleted_at,labels,tiers\r\n,[email protected],Sam Memberino,Early supporter,,,,,,,`;
16+
const expected = `id,email,name,note,subscribed_to_emails,complimentary_plan,stripe_customer_id,created_at,deleted_at,labels,tiers\r\n,[email protected],Sam Memberino,Early supporter,false,,,,,,`;
1717
assert.equal(result, expected);
1818
});
1919

2020
it('maps the subscribed property to subscribed_to_emails', function () {
2121
const json = [{
2222
23-
subscribed: false
23+
subscribed_to_emails: false
2424
}];
2525

2626
const columns = [

0 commit comments

Comments
 (0)