-
Notifications
You must be signed in to change notification settings - Fork 168
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
MWPW-154059 - Milo templates library #2855
Conversation
* Allows template-metadata block to output metadata block in template * Fixes inital listing order * Adds search ability to templates Resolves: MWPW-154059
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## stage #2855 +/- ##
==========================================
+ Coverage 95.91% 96.24% +0.33%
==========================================
Files 173 236 +63
Lines 45831 54233 +8402
==========================================
+ Hits 43958 52196 +8238
- Misses 1873 2037 +164 ☔ View full report in Codecov by Sentry. |
Will look into unit tests. |
This pull request is not passing all required checks. Please see this discussion for information on how to get all checks passing. Inconsistent checks can be manually retried. If a test absolutely can not pass for a good reason, please add a comment with an explanation to the PR. |
@@ -32,27 +32,47 @@ async function loadPersonalization(content, list) { | |||
personalization(content, list); | |||
} | |||
|
|||
function addSearch(content, list) { | |||
function addSearch(content, list, type) { |
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.
Same as above. I think it would be good to pass an object.
function addSearch(content, list, type) { | |
function addSearch({ content, list, type }) { |
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.
Done
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.
LGTM. I tested and everything works as expected.
Reminder to set the |
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.
Verified , ready for STAGE
Testing details https://jira.corp.adobe.com/browse/MWPW-154059
template-metadata
block to be pasted asmetadata
block in templateResolves: MWPW-154059
Test URLs:
BACOM Test URLs:
Testing notes:
With the updated features, the template named "demo" has search tags added (Rc-special demo test) via of a
library-metadata
block. This block allows authors to add custom search tags to the template. Searching with any of those tags will pull up the "Demo" template. However, when the template is copied from the library and pasted to a Word doclibrary-metadata
block should not be part of the copied blocks.Testing this using the
main
branch will copy thelibrary-metadata
block to the Word doc.To include a
metadata
block in the template, to be pasted into the Word doc, the template must include atemplate-metadata
block with all the needed metadata fields. Once, pasted in the page that block converted tometadata
and displays as a metadata block.If a
library-metadata
block with the key value pair (searchTags
and search words) is added to a template, that template will be searchable by the name of the template along with the search words added tolibrary-metadata
.Block searching should continue to work as it's always worked.