TryGhost/Ghost
1
0
Fork 0
mirror of https://github.com/TryGhost/Ghost.git synced 2024-11-28 05:37:34 +03:00
Code Issues Projects Releases Wiki Activity

Added newsletter permissions for Editors and Authors

Browse Source 
refs https://github.com/TryGhost/Team/issues/1618

- Editors and Authors require read access to newsletters as part of the publish + send workflow
This commit is contained in:
Matt Hanley 2022-05-12 11:32:30 +01:00 committed by Matt Hanley
parent 78d1132b13
commit 1fabd76391
6 changed files with 37 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
core/server/data/migrations/versions/5.0/2022-05-12-10-29-add-newsletter-permissions-for-editors-and-authors.js Normal file
View File

@ -0,0 +1,20 @@
const {combineTransactionalMigrations, addPermissionToRole} = require('../../utils');
module.exports = combineTransactionalMigrations(
addPermissionToRole({
permission: 'Browse newsletters',
role: 'Editor'
}),
addPermissionToRole({
permission: 'Read newsletters',
role: 'Editor'
}),
addPermissionToRole({
permission: 'Browse newsletters',
role: 'Author'
}),
addPermissionToRole({
permission: 'Read newsletters',
role: 'Author'
})
);

6
core/server/data/schema/fixtures/fixtures.json
View File

@ -723,7 +723,8 @@
"email": "all", "email": "all",
"snippet": "all", "snippet": "all",
"label": ["browse", "read"], "label": ["browse", "read"],
"product": ["browse", "read"] "product": ["browse", "read"],
"newsletter": ["browse", "read"]
}, },
"Author": { "Author": {
"post": ["browse", "read", "add"], "post": ["browse", "read", "add"],
@ -736,7 +737,8 @@
"email_preview": "read", "email_preview": "read",
"email": "read", "email": "read",
"snippet": ["browse", "read"], "snippet": ["browse", "read"],
"product": ["browse", "read"] "product": ["browse", "read"],
"newsletter": ["browse", "read"]
}, },
"Contributor": { "Contributor": {
"post": ["browse", "read", "add"], "post": ["browse", "read", "add"],

4
test/integration/migrations/migration.test.js
View File

@ -163,8 +163,8 @@ describe('Database Migration (special functions)', function () {
permissions.should.havePermission('Browse custom theme settings', ['Administrator']); permissions.should.havePermission('Browse custom theme settings', ['Administrator']);
permissions.should.havePermission('Edit custom theme settings', ['Administrator']); permissions.should.havePermission('Edit custom theme settings', ['Administrator']);
permissions.should.havePermission('Browse newsletters', ['Administrator', 'Admin Integration']); permissions.should.havePermission('Browse newsletters', ['Administrator', 'Editor', 'Author', 'Admin Integration']);
permissions.should.havePermission('Read newsletters', ['Administrator', 'Admin Integration']); permissions.should.havePermission('Read newsletters', ['Administrator', 'Editor', 'Author', 'Admin Integration']);
permissions.should.havePermission('Edit newsletters', ['Administrator', 'Admin Integration']); permissions.should.havePermission('Edit newsletters', ['Administrator', 'Admin Integration']);
permissions.should.havePermission('Add newsletters', ['Administrator', 'Admin Integration']); permissions.should.havePermission('Add newsletters', ['Administrator', 'Admin Integration']);
}); });

12
test/unit/server/data/schema/fixtures/fixture-manager.test.js
View File

@ -201,18 +201,18 @@ describe('Migration Fixture Utils', function () {
fixtureManager.addFixturesForRelation(fixtures.relations[0]).then(function (result) { fixtureManager.addFixturesForRelation(fixtures.relations[0]).then(function (result) {
should.exist(result); should.exist(result);
result.should.be.an.Object(); result.should.be.an.Object();
result.should.have.property('expected', 86); result.should.have.property('expected', 88);
result.should.have.property('done', 86); result.should.have.property('done', 88);
// Permissions & Roles // Permissions & Roles
permsAllStub.calledOnce.should.be.true(); permsAllStub.calledOnce.should.be.true();
rolesAllStub.calledOnce.should.be.true(); rolesAllStub.calledOnce.should.be.true();
dataMethodStub.filter.callCount.should.eql(86); dataMethodStub.filter.callCount.should.eql(88);
dataMethodStub.find.callCount.should.eql(7); dataMethodStub.find.callCount.should.eql(7);
baseUtilAttachStub.callCount.should.eql(86); baseUtilAttachStub.callCount.should.eql(88);
fromItem.related.callCount.should.eql(86); fromItem.related.callCount.should.eql(88);
fromItem.find.callCount.should.eql(86); fromItem.find.callCount.should.eql(88);
done(); done();
}).catch(done); }).catch(done);

2
test/unit/server/data/schema/integrity.test.js
View File

@ -36,7 +36,7 @@ const validateRouteSettings = require('../../../../../core/server/services/route
describe('DB version integrity', function () { describe('DB version integrity', function () {
// Only these variables should need updating // Only these variables should need updating
const currentSchemaHash = '2f4266e6e5087ad92dd30f3e721d46e5'; const currentSchemaHash = '2f4266e6e5087ad92dd30f3e721d46e5';
const currentFixturesHash = 'f4795020369ec3a770b538be8d8b2536'; const currentFixturesHash = '2219972fb91a30f8d740e05afd3a033c';
const currentSettingsHash = 'ffd899a82b0ad2886e92d8244bcbca6a'; const currentSettingsHash = 'ffd899a82b0ad2886e92d8244bcbca6a';
const currentRoutesHash = '3d180d52c663d173a6be791ef411ed01'; const currentRoutesHash = '3d180d52c663d173a6be791ef411ed01';

6
test/utils/fixtures/fixtures.json
View File

@ -896,7 +896,8 @@
"email": "all", "email": "all",
"snippet": "all", "snippet": "all",
"label": ["browse", "read"], "label": ["browse", "read"],
"product": ["browse", "read"] "product": ["browse", "read"],
"newsletter": ["browse", "read"]
}, },
"Author": { "Author": {
"post": ["browse", "read", "add"], "post": ["browse", "read", "add"],
@ -909,7 +910,8 @@
"email_preview": "read", "email_preview": "read",
"email": "read", "email": "read",
"snippet": ["browse", "read"], "snippet": ["browse", "read"],
"product": ["browse", "read"] "product": ["browse", "read"],
"newsletter": ["browse", "read"]
}, },
"Contributor": { "Contributor": {
"post": ["browse", "read", "add"], "post": ["browse", "read", "add"],