Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Living social is alive #6

Merged
merged 4 commits into from

2 participants

@jonathonloo
Collaborator

Small simple fix to enable the title to correctly show for Living Social. Forgive the multiple commits. Despite squashing and trying to clean it up locally the remote repo didnt take too kindly to my "organization". Still a github n00b.

@austenito austenito merged commit 15c01a8 into hicapacity:master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
This page is out of date. Refresh to see the latest.
Showing with 5 additions and 1 deletion.
  1. +5 −1 background.html
View
6 background.html
@@ -118,7 +118,7 @@
crawl_json_feed(url,
function(response) {
var entries = response.responseData.feed.entries;
- store_deal(deal_source, entries);
+ store_deal(deal_source, entries);
}
);
})(deal_source, url);
@@ -137,6 +137,10 @@
var title = entries[index].title;
var link = entries[index].link;
+ if(deal_source == 'Living Social')
+ {
+ title = entries[index].contentSnippet;
+ }
// If the deal doesn't exist, update the browser icon.
if (deal_titles.indexOf(title) == -1) {
chrome.browserAction.setIcon({"path" : "images/dealios22.png"});
Something went wrong with that request. Please try again.