[MIG] website_local_font: Migration to 16.0#1028
[MIG] website_local_font: Migration to 16.0#1028Reyes4711-S73 wants to merge 3 commits intoOCA:16.0from
Conversation
[ADD] website_local_fonts: add new module [ADD] website_local_fonts: add new module Updated Test Folder Name Changes as per suggestions in the PR Added missing setup file
| ), | ||
| ], | ||
| "web.assets_backend": [ | ||
| "website_local_font/static/src/xml/website.editor.xml", |
There was a problem hiding this comment.
Can you add this xml file to website.assets_wysiwyg asset instead of website_assets_backend? That way you don't need to define this in the snippets.options.js as xmldependency
There was a problem hiding this comment.
snippets.options.js.txt
Modify the code as shared in the attached file. I have removed xmlDependencies, inherited the start method in a way that it can be inherited by other modules in future, removed widgetObj in _saveLocalFont function and replaced by this and made use of "const" instead of "var"
Reyes4711-S73
force-pushed
the
16.0-mig-website_local_fonts-new
branch
2 times, most recently
from
2ce6be5 to
3ab6ef1
Compare
|
@ByteMeAsap I put your changes but, I tested in a local environment and in a runboat and I have this error. |
Reyes4711-S73
force-pushed
the
16.0-mig-website_local_fonts-new
branch
from
3ab6ef1 to
e9cad2a
Compare
|
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
github-actions
bot
added
the
stale
3 participants
Standard migration to 16.0
Reopen from #999