Skip to content

Ports - Laneia#53

Open
laneia wants to merge 97 commits intoAda-C11:masterfrom
laneia:master
Open

Ports - Laneia#53
laneia wants to merge 97 commits intoAda-C11:masterfrom
laneia:master

Conversation

@laneia
Copy link

@laneia laneia commented Jun 18, 2019

Media Ranker

Congratulations! You're submitting your assignment!

Comprehension Questions

Question Answer
Describe a custom model method you wrote. I wrote a list_of method that utilizes a sort_by_votes method that returns an array of works in each category, sorted first by number of votes and then alphabetically.
Describe how you approached testing that model method. What edge cases did you come up with? I first tested expected situations like making sure the method would only return works of a single category. Then I looked at edge cases like if it returned an empty array if there are no works and that the vote count worked if there were no votes.
What are session and flash? What is the difference between them? Session is an object that stores user data for the duration of the session (until the window is closed). Flash is very similar but the data doesn't persist beyond the one request-response cycle.
What was one thing that you gained more clarity on through this assignment? I learned a lot about how to utilize session for various purposes like making sure a user is logged in to vote.
What is the Heroku URL of your deployed application? https://laneia-media-ranker.herokuapp.com/
Do you have any recommendations on how we could improve this project for the next cohort? Include some movies in the seed data.

laneia added 30 commits June 14, 2019 12:39
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