Skip to content
This repository has been archived by the owner on Oct 15, 2022. It is now read-only.

Change some carousels to work with the new API #504

Merged
merged 6 commits into from Apr 8, 2014
Merged

Change some carousels to work with the new API #504

merged 6 commits into from Apr 8, 2014

Conversation

jagtalon
Copy link
Member

@jagtalon jagtalon commented Apr 8, 2014

Barely working instant answers. @russellholt

@jagtalon
Copy link
Member Author

jagtalon commented Apr 8, 2014

  • alternative_to/alternative_to.js
  • bbc/bbc.js
  • bootic/bootic.js
  • coupon_mountain/coupon_mountain.js
  • forecast/forecast.js (skipped--already works)
  • game_info/game_info.js (skipped--not yet live)
  • google_plus/google_plus.js
  • guidebox/getid/guidebox_getid.js
  • in_theaters/in_theaters.js (skipped--already works)
  • khan_academy/khan_academy.js (skipped--video works just as well)
  • octopart/octopart.js (skipped--already converted)
  • sound_cloud/sound_cloud.js

russellholt added a commit that referenced this pull request Apr 8, 2014
@russellholt
Merge pull request #504 from duckduckgo/jag/bttfbeta/carousel
ed4199c 
Change some carousels to work with the new API
@russellholt russellholt merged commit ed4199c into bttf-beta Apr 8, 2014
@russellholt russellholt deleted the jag/bttfbeta/carousel branch April 8, 2014 20:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@jagtalon @russellholt