Skip to content
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

Clean up policy templates #5502

Closed
wants to merge 10 commits into from
Closed

Clean up policy templates #5502

wants to merge 10 commits into from

Conversation

Rotonen
Copy link
Contributor

@Rotonen Rotonen commented Mar 14, 2019

🚧 Depends on 4teamwork/gever-buildouts#30

  • Cleaned up old cruft
  • Now matches the default sold SaaS configuration

Closes #5409

@Rotonen Rotonen added this to the Release 2019.2 milestone Mar 14, 2019
@Rotonen Rotonen requested a review from a team March 14, 2019 15:56
Copy link
Contributor

@deiferni deiferni left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks 👍

Two more things up for discussion. Have any of our SaaS customers actually disabled these two:

  • private folder
  • dossier template

Otherwise we could eliminate the feature alltogether and flip the default if necessary.

@@ -87,7 +87,7 @@ setup.enable_meeting_feature.post_ask_hook = mrbob.hooks:to_boolean

setup.enable_docproperty_feature.question = Enable docproperty feature
setup.enable_docproperty_feature.required = True
setup.enable_docproperty_feature.default = False
setup.enable_docproperty_feature.default = True
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we should discuss to flip the default in ITemplateFolderProperties and get rid of this question alltogether.

@Rotonen
Copy link
Contributor Author

Rotonen commented Jun 11, 2019

@phgross this depends on us figuring out how to gain enough confidence to get 4teamwork/gever-buildouts#30 merged.

@phgross
Copy link
Member

phgross commented Nov 6, 2019

Outdated, as discussed in the daily, I close this PR .

@phgross phgross closed this Nov 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ensure Solr is included for new SaaS deployments
3 participants