rss-reader sample app should not include google api key

Bug #1224051 reported by Kyle Nitzsche
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu HTML5 UI SDK
Fix Released
Low
Adnane Belmadiaf
ubuntu-html5-theme (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

Using:
ubuntu-html5-theme-examples 0.1+13.10.20130805bzr76raring0

This package installs rss-reader sample app here:
/usr/share/ubuntu-html5-theme/0.1/examples/apps/rss-reader/*

Including index.html which includes this specific google api key:
cript type="text/javascript" src="https://www.google.com/jsapi?key=ABQIAAAAuvAXPRXL9qkzO0R8vzYpOhQ_BAnb8hQeSCQ2aBH79hHa1YbW_RQNN9ClKXG21xQObiXJ7diAAYaywg"></script>

I believe no key is needed, according to this:
https://developers.google.com/feed/v1/devguide#load_the_javascript_api_and_ajax_search_module

When I change to no key like this, the app still works as expected:
    <script type="text/javascript" src="https://www.google.com/jsapi"></script>

Related branches

Adnane Belmadiaf (daker)
summary: - rrs-reader sample app should not include google api key
+ rss-reader sample app should not include google api key
Adnane Belmadiaf (daker)
Changed in ubuntu-html5-theme:
status: New → Triaged
importance: Undecided → Low
assignee: nobody → Adnane Belmadiaf (daker)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:ubuntu-html5-theme at revision 79, scheduled for release in ubuntu-html5-theme, milestone Unknown

Changed in ubuntu-html5-theme:
status: Triaged → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-html5-theme - 0.1+13.10.20130916-0ubuntu1

---------------
ubuntu-html5-theme (0.1+13.10.20130916-0ubuntu1) saucy; urgency=low

  [ daker ]
  * Fixed the popover position when it's used with the footer. (LP:
    #1216307)
  * Removed the api key since it's not needed. (LP: #1224051)
  * Fixed the Switchs CSS to match the QML theme. (LP: #1224188)

  [ Alexandre Abreu ]
  * The ui-gallery app was not being installed Add simple desktop file.
    (LP: #1221920)
  * Add qml launcher for the ui-gallery to have a more consistent view
    of the widgets.

  [ Ubuntu daily release ]
  * Automatic snapshot from revision 81
 -- Ubuntu daily release <email address hidden> Mon, 16 Sep 2013 11:34:53 +0000

Changed in ubuntu-html5-theme (Ubuntu):
status: New → Fix Released
Adnane Belmadiaf (daker)
Changed in ubuntu-html5-theme:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.