ylqjgm
/
canvas-lms
mirror of https://github.com/instructure/canvas-lms.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Update ToolConfiguration Schema 

Some fields should not be required, fix typo
in Deeplinkingrequest.

fixes PLAT-4284

Test Plan:
 - n/a

Change-Id: I36c3c84a143d8aaff7ddb9de7c33847640a6845a
Reviewed-on: https://gerrit.instructure.com/184627
Tested-by: Jenkins
Product-Review: Marc Phillips <mphillips@instructure.com>
Reviewed-by: Weston Dransfield <wdransfield@instructure.com>
QA-Review: Weston Dransfield <wdransfield@instructure.com>
This commit is contained in:
Marc Phillips 2019-03-08 15:13:10 -07:00
parent 7f02574274
commit 14ae14dfaa
1 changed files with 9 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
lib/schemas/lti/tool_configuration.rb
View File

@ -71,10 +71,6 @@ module Schemas::Lti
"settings" => {
"type" => "object",
"required" => [
"text",
"icon_url",
"selection_width",
"selection_height",
"placements"
].freeze,
"properties" => {
@ -89,10 +85,7 @@ module Schemas::Lti
"items" => {
"type" => "object",
"required" => [
"placement",
"target_link_uri",
"enabled",
"message_type"
"placement"
].freeze,
"properties" => {
"placement" => {
@ -137,7 +130,7 @@ module Schemas::Lti
"message_type" => {
"type" => "string",
"enum" => [
"LtiDeeplLinkingRequest",
"LtiDeepLinkingRequest",
"LtiResourceLinkRequest"
].freeze
}.freeze,
@ -146,7 +139,13 @@ module Schemas::Lti
"enum" => [
"icon-lti"
].freeze
}.freeze
}.freeze,
"selection_width" => {
"type" => "number"
}.freeze,
"selection_height" => {
"type" => "number"
}.freeze,
}.freeze
}.freeze
}.freeze