Skip to content
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

Add search to add course / session learning material #912

Merged
merged 1 commit into from
Oct 23, 2015

Conversation

jrjohnson
Copy link
Member

Allows the user to find an already created learning material and then
add it to a course or session. Any LMs that have already been added
are excluded from the results.

We probably need to add some ore context to the results, that can be done later though.

Requires ilios/ilios#1055 to work with the API. I did add the required endpoints to the dev data so it will work without a proxy for limited testing. The keywords 'internal' and 'dev' produce results.

Fixes #389

@@ -267,7 +267,7 @@ export default {
'copyrightPermission': 'Copyright Permission',
'copyrightAgreement': "The file I am attempting to upload is my own or I have express permission to reproduce and/or distribute this item and does not contain any protected health information. My use of this file is in compliance with Government and University policies on copyright and information security and my educational program's guidelines for professional conduct. This file also adheres to the Terms and Conditions for this application.",
'copyrightRationale':'Copyright Rationale',

'searchPlaceholder': 'Find Learning Material',
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@saschaben / @dartajax please translate. 'Find Learning Material'

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Encontrar Material de Aprendizaje

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Trouver matières d'étude

Allows the user to find an already created learning material and then
add it to a course or session.  Any LMs that have already been added
are excluded from the results.
@jrjohnson
Copy link
Member Author

This is ready for code review and merge.

thecoolestguy added a commit that referenced this pull request Oct 23, 2015
Add search to add course / session learning material
@thecoolestguy thecoolestguy merged commit 0e353c5 into ilios:master Oct 23, 2015
@jrjohnson jrjohnson deleted the 389-lmsearch branch October 23, 2015 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants