-
Notifications
You must be signed in to change notification settings - Fork 10
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
Dev/2.1.0 #74
Dev/2.1.0 #74
Conversation
clpetersonucf
commented
Jul 19, 2019
•
edited
Loading
edited
- Fixes default collaborator selection instead of a mysterious blank option (#1231)
- Fixes export options screen error when semester list is empty (#1243)
- Fixes formatting for long titles in LTI widget picker (#1129)
- Relocates logged in user in the header (Move the currently logged in user display to the right hand side of the page #64)
- Adds export options for referrer URLs (#176)
- Fixes errors caused by unfiltered postMessages (Materia.Engine does not check if received POST messages are valid #78)
- Adds clarifying information to Export Options modal concerning LTI launches (#1216)
- Minor styling tweaks to guide page iframes
- Support for participation score logging (#1217)
- Removes "group" field for widgets (issue/707 - remove "group" field #73)
Bumps [lodash](https://github.com/lodash/lodash) from 4.17.11 to 4.17.14. - [Release notes](https://github.com/lodash/lodash/releases) - [Commits](lodash/lodash@4.17.11...4.17.14) Signed-off-by: dependabot[bot] <[email protected]>
…on-level issue/1158 - default collaboration level
…17.14 Bump lodash from 4.17.11 to 4.17.14
…property #1243 TypeError: Cannot set property 'checked' of undefined when selecting export options on widget with no scores
#1129 lti picker doesn't handle long titles
issue/64-move-logged-in-user
issue/176 - Referrer Urls
#78 Materia.Engine does not check if received POST messages are valid
#1216 Inline help on my widgets to inform users of integration options
…for-helper-docs CSS tweaks to helper docs frame
…on-scores issue/1217 - submit participation score
…dget issue/707 - remove "group" field
Ready for review. |
This one looks good to me, tested on QA in conjunction with the dev/6 branch and everything works. Needs approval. |
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.
Found some potential issues in the LTI picker that could stand to be resolved, but they're outside the scope of these changes and can get their own issue.
Everything here looks like it's functioning well.
All looks good, though I have feedback on ucfopen/Materia#1245 that'll apply here |
5830590