-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
NV-1541: handle default novu layout #2551
Conversation
NV-1541 Handle default novu layout
create default layout per organization + environment. show it in the layouts table. As editable/deletable? if no layout - compile novuDefaultLayout only to editor |
"include": "app/content-templates/usecases/compile-template/templates/layout.handlebars", | ||
"include": "app/layouts/usecases/get-novu-layout/templates/layout.handlebars", | ||
"outDir": "dist/src" | ||
}, | ||
{ | ||
"include": "app/content-templates/usecases/compile-template/templates/basic.handlebars", | ||
"include": "app/content-templates/usecases/compile-email-template/templates/basic.handlebars", | ||
"outDir": "dist/src" |
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.
Refactored the locaiton of those to live closely to the relevant parts where they belong to
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.
Some comments. Thanks for helping us with this.
What change does this PR introduce?
When creating a new environment add default layout to it.
Why was this change needed?
Closes NV-1541 to make sure new organizations created are created with the default novu layout.
Other information (Screenshots)