Skip to content

Commit cf53c45

Browse files
author
Eric Berry
committed
Correct defaults for app center plugin
fixes PLAT-806 Change-Id: Iec9cb69c80fab471b1aff4d8b8d7bc2dc4190520 Reviewed-on: https://gerrit.instructure.com/45783 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins <jenkins@instructure.com> QA-Review: Eric Berry <ericb@instructure.com> Product-Review: Eric Berry <ericb@instructure.com>
1 parent 044637b commit cf53c45

3 files changed

Lines changed: 7 additions & 12 deletions

File tree

app/views/plugins/_app_center_settings.html.erb

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -2,14 +2,9 @@
22
<table style="width: 500px;" class="formtable">
33
<tr>
44
<td colspan="2">
5-
<p><%= t(:description, <<-TEXT)
6-
App Center will appear in the "Apps" tab on the setings
7-
page of either a course or account. The URL provided below should
8-
point to a valid App Center endpoint
9-
(for example, https://lti-examples.heroku.com -- be sure to omit
10-
the trailing slash)
11-
TEXT
12-
%></p>
5+
<p><%= t "App Center will appear in the \"Apps\" tab on the settings page of either a course or account. " +
6+
"The URL provided below should point to a valid App Center endpoint (for example, %{url} -- be sure to omit the trailing slash",
7+
url: "https://www.eduappcenter.com" %></p>
138
</td>
149
</tr>
1510
<tr>

db/migrate/20120505003400_add_tool_id_to_external_tools.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ class AddToolIdToExternalTools < ActiveRecord::Migration
22
tag :predeploy
33
def self.up
44
# using tool_id instead of developer_key.id lets us
5-
# use the same keys as lti-examples.heroku.com for
5+
# use the same keys as www.eduappcenter.com for
66
# tying multiple context_external_tools to the
77
# same third-party tool
88
add_column :context_external_tools, :tool_id, :string

lib/canvas/plugins/default_plugins.rb

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -286,10 +286,10 @@
286286
:description => lambda{ t :description, 'App Center for tracking/installing external tools in Canvas' },
287287
:settings_partial => 'plugins/app_center_settings',
288288
:settings => {
289-
:base_url => 'https://www.edu-apps.org',
289+
:base_url => 'https://www.eduappcenter.com',
290290
:token => nil,
291-
:apps_index_endpoint => '/api/v1/apps',
292-
:app_reviews_endpoint => '/api/v1/apps/:id/reviews'
291+
:apps_index_endpoint => '/api/v1/lti_apps',
292+
:app_reviews_endpoint => '/api/v1/lti_apps/:id/reviews'
293293
},
294294
:validator => 'AppCenterValidator'
295295
})

0 commit comments

Comments
 (0)