[ENG-6102] Added context phrase to institutions filter modal on search pages #2514
Conversation
Pull Request Test Coverage Report for Build 13460664502Details
💛 - Coveralls |
lib/osf-components/addon/components/search-page/filter-facet/component.ts
Outdated
Show resolved
Hide resolved
lib/osf-components/addon/components/search-page/filter-facet/component.ts
Outdated
Show resolved
Hide resolved
lib/osf-components/addon/components/search-page/filter-facet/template.hbs
Outdated
Show resolved
Hide resolved
| {{#if (and this.hasDescription)}} | ||
| {{ this.description }} <a href={{this.link}}> {{ this.link_text }} </a> | ||
| {{else }} | ||
| {{t 'search.filter-facet.see-more-modal-text'}} | ||
| {{/if}} |
There was a problem hiding this comment.
Question: Does the PropertyPath.description field from the SHARE API return the entire phrase "Please select a filter to apply to your search. Only OSF Institutions member affiliations are discoverable."?
Do we want the SHARE API only return "Only OSF Institutions member affiliations..." in the PropertyPath.description field and the FE just shows "Please select a filter to apply to your search." no matter what?
There was a problem hiding this comment.
Yeah, I designed SHARE API to return description to return Please select an institution to apply to your search. Only OSF Institutions member affiliations are discoverable., linkText to return text that represents a link and link itselt.
On the ticket description, the context phrase is different from the current phrase on the Frontend (Please select a filter to apply to your search), so I added the part Please select an institution to apply to your search to the description from SHARE API. I can change the phrases that SHARE API returns if needed.
Purpose
Added context phrase to institutions filter modal on search pages
Summary of Changes
TBD
Screenshot(s)
Side Effects
TBD
QA Notes
TBD