Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@aldeed
Copy link
Contributor

@aldeed aldeed commented Aug 7, 2019

Impact: breaking
Type: feature

Part of epic #4996

Issue

There is startup code that loads template assets from /private. This feature isn't used in 2.x because only emails currently use templates, and they are loaded from code files by the email-templates plugin.

Solution

Because loading these static assets is a Meteor feature, we've removed this startup code.

Breaking changes

The following are removed:

  • Reaction.registerTemplate
  • Automatic SimpleSchema validation on Templates collection

Testing

  • Verify app starts
  • Verify viewing and editing templates on "Settings > Template" page of Reaction Admin works as before.

@aldeed aldeed requested a review from kieckhafer August 7, 2019 04:24
@aldeed aldeed self-assigned this Aug 7, 2019
@aldeed aldeed mentioned this pull request Aug 7, 2019
45 tasks
Copy link
Member

@kieckhafer kieckhafer left a comment

Choose a reason for hiding this comment

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

👍 Looks good, app works as expected, no changes to editing templates.

@kieckhafer kieckhafer merged commit 48617a3 into develop Aug 7, 2019
@kieckhafer kieckhafer deleted the feat-remove-startup-template-import branch August 7, 2019 18:50
@kieckhafer kieckhafer mentioned this pull request Aug 28, 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.

3 participants