-
Notifications
You must be signed in to change notification settings - Fork 188
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
Implement recommendations display #4803
Open
akolson
wants to merge
39
commits into
learningequality:search-recommendations
Choose a base branch
from
akolson:implement-recommendations-display
base: search-recommendations
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 37 commits
Commits
Show all changes
39 commits
Select commit
Hold shift + click to select a range
23e49e6
Adds translator object + strings for search recs feature
akolson 3e85347
change SearchOrBrowserWindow from Vuetify to KDS components
akolson fe76812
add about recommendations link
akolson 1b5fe71
Update Translator object to expose a destructurable interface for str…
akolson 13c7d15
Merge branch 'search-recs-strings-feature-file' into implement-recomm…
akolson f51e08f
adds recommendations title
akolson eeb8537
updates strings
akolson 003d109
Merge branch 'search-recs-strings-feature-file' into implement-recomm…
akolson a43dcbb
Adds about recommendations modal
akolson 675cdae
updates strings
akolson c56f907
Merge branch 'search-recs-strings-feature-file' into implement-recomm…
akolson 3532e75
Adds reponsiveness to page
akolson c8f8db9
Adds sample data request
akolson fa81206
Merge branch 'merge-into-search-recs' into implement-recommendations-…
akolson 3a74adc
Add latest KDS rc4 as dependency
akolson 835e78c
Adds KCard
akolson b80aedb
Merge branch 'search-recommendations' into implement-recommendations-…
akolson e3f0cba
Adds KCardGrid
akolson 93b8ef3
Adds preliminary logic to load recommendations
akolson 078638c
Merge remote-tracking branch 'upstream/unstable' into merge-into-sear…
akolson 4f1f84e
Merge branch 'merge-into-search-recs' into implement-recommendations-…
akolson 1d000da
Merge branch 'merge-into-search-recs' into implement-recommendations-…
akolson 8e200d5
fixes bug in translateMetadataString
akolson dc29890
Make minor tweaks to UI
akolson 7009c10
updates kds version
akolson 30d397f
Adds pagination logic
akolson 7e98efc
Refactors code
akolson e4e2eeb
More code refators and logic fixes
akolson 4ce5884
code clean-up
akolson 95a1b52
adds ai flag check on UI and fetch
akolson 0cb9469
Merge remote-tracking branch 'upstream/search-recommendations' into i…
akolson 9f8de26
minor UI fixes
akolson 596e109
further UI improvements
akolson 7574c06
UI fixes
akolson c6e3a0c
additional UI fixes
akolson 75df365
Updates kds to latest version
akolson 48cda0d
Merge branch 'search-recommendations' into implement-recommendations-…
akolson 47725f3
Add todo to relax an unlikely situation where channel names will be null
akolson 6b85c8d
Removes ambiguous selected proo from recommended resource card
akolson File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
Oops, something went wrong.
Oops, something went wrong.
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.
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.
Would there ever not be a channel name?
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.
Most likely not. If I can recall correctly, this change was in response to a bug I was experience while simulating the recommendations while using Studio's local app data. For some reason, some the the resources returned no channel names. I'll add a comment so this can be removed once the integration is complete.