Refactor from EDS profiles to EDS Facets #78
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Coverage remained the same at 100.0% when pulling a26f2b1831617bb1d7159db5f922ecf997813322 on di136_facetize into 4ff7499 on master. |
|
Coverage remained the same at 100.0% when pulling a26f2b1831617bb1d7159db5f922ecf997813322 on di136_facetize into 4ff7499 on master. |
|
Coverage remained the same at 100.0% when pulling e2bada5fc90a7a9b034430671fe6fb8d548b3a70 on di136_facetize into 4ff7499 on master. |
JPrevost
force-pushed
the
di136_facetize
branch
from
e2bada5 to
26c2a5e
Compare
|
Coverage remained the same at 100.0% when pulling 26c2a5e01c56ded06f72e26ee9b67ae288b2607a on di136_facetize into 4ff7499 on master. |
What: * Moves from multiple EDS Admin configured profiles to a faceted approach that uses the source facet to segregate content into appropriate boxes Why: * EDS profiles would never allow us to create the boxes we were asked to create because we were basing the boxes on format of the material and profiles can't be created effective that way. * https://mitlibraries.atlassian.net/browse/DI-136 How: * Facets for the (currently 3) boxes are configured via the ENV settings. EDS_ARTICLE_FACETS, EDS_BOOK_FACETS, EDS_WHATNOT_FACETS Note: * The link to more like this is temporarily disabled as we look into how to deep link into pre-faceted result sets in EDS UI. That issue is tracked via https://mitlibraries.atlassian.net/browse/DI-143
JPrevost
force-pushed
the
di136_facetize
branch
from
26c2a5e to
8661217
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What:
approach that uses the source facet to segregate content into
appropriate boxes
Why:
to create because we were basing the boxes on format of the material
and profiles can't be created effective that way.
How:
EDS_ARTICLE_FACETS, EDS_BOOK_FACETS, EDS_WHATNOT_FACETS
Note:
deep link into pre-faceted result sets in EDS UI. That issue is tracked
via https://mitlibraries.atlassian.net/browse/DI-143