Skip to content

Final PR for SeeMore Project - #23

Open
brittinator wants to merge 438 commits into
Ada-Developers-Academy:FeedMefrom
brittinator:FeedMe
Open

Final PR for SeeMore Project#23
brittinator wants to merge 438 commits into
Ada-Developers-Academy:FeedMefrom
brittinator:FeedMe

Conversation

@brittinator

Copy link
Copy Markdown

Our site is simple and clean. We have quite a few validations to make sure the incoming data will not break the site. We also did a couple big refactors where we moved methods from the controllers and models to API helper websites.

Known issues:

  • when subscribing to a private twitter user and refreshing newfeed this breaks the site.
  • Conscious decision when logging out of our site does not log you out of instagram or github.

Known bug from instagram: sometimes pulls 3 users from the search page as it's supposed to; other times pulls 4 users.

There may be an easter egg if you try to login to the heroic website using the developer's strategy...

a-lmx and others added 30 commits August 10, 2015 15:54
…ce the last_post_id (or the last 5 tweets if there is no last_post_id for a followee). We also created a find_twitter_params method to pull necessary API info into a hash, which we use ot Post.create.
wrote find_instagram_params, fixed name of method in call from find_twitter_params to find_instagram_params
added logic to newsfeed to create variables for display for debugging
knguyen0710 and others added 29 commits August 13, 2015 09:03
catchingash pushed a commit that referenced this pull request Feb 25, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants