Skip to content

Code Challenge - Jared Murphy#12

Open
jaredmurphy wants to merge 5 commits intobarkbox:masterfrom
jaredmurphy:master
Open

Code Challenge - Jared Murphy#12
jaredmurphy wants to merge 5 commits intobarkbox:masterfrom
jaredmurphy:master

Conversation

@jaredmurphy
Copy link

@jaredmurphy jaredmurphy commented Aug 26, 2020

👋 I completed 4 of the 7 challenge prompts as well as started sorting the index page by the number of likes in the last hour. Due to time constraints, I was not able to sort by likes in the last hour but was able to sort by the number of overall likes.

It was fun to figure out how to inject an ad after every 2nd dog since I hadn't done this before, and making the Like/Unlike functionality async with Unobtrusive JS. If I had more time I would have completed the sorting prompt by only considering likes in the last hour will sorting, and adding more tests.

  • Add pagination to index page, to display 5 dogs per page
  • Add the ability to for a user to input multiple dog images on an edit form or new dog form
  • Associate dogs with owners
  • Allow editing only by owner
  • Allow users to like other dogs (not their own)
  • Allow sorting the index page by number of likes in the last hour - STARTED - see comment in app/models/dog.rb
  • Display the ad.jpg image (saved at app/assets/images/ad.jpg) after every 2 dogs in the index page, to simulate advertisements in a feed

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.

1 participant