-
Notifications
You must be signed in to change notification settings - Fork 598
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PRO-7053: enable document-versions for autopublished docs #4851
Conversation
@@ -5,6 +5,7 @@ module.exports = { | |||
pluralLabel: 'apostrophe:fileTags', | |||
quickCreate: false, | |||
autopublish: true, | |||
versions: true, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah... we could do this in an improvement in the document-versions module, but this is a lot less typing and harms nothing when it is not present. 👍
@@ -80,6 +80,7 @@ export default { | |||
|
|||
if (modal) { | |||
await apos.modal.execute(modal, { | |||
moduleName: this.moduleOptions.name, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So you can look up options of it, OK
…053-document-versions-autopublish
@@ -48,6 +48,7 @@ module.exports = { | |||
searchable: false, | |||
slugPrefix: 'user-', | |||
localized: false, | |||
versions: false, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@boutell added this as an extra security
Summary
See: https://github.com/apostrophecms/document-versions/pull/73
And:
https://github.com/apostrophecms/palette/pull/110
apostrophecms/import-export#102
Cypress tests:
https://github.com/apostrophecms/testbed/pull/323
What kind of change does this PR introduce?
(Check at least one)
Make sure the PR fulfills these requirements:
If adding a new feature without an already open issue, it's best to open a feature request issue first and wait for approval before working on it.
Other information: