Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Yasmin - Media Ranker Project #95

Open
wants to merge 42 commits into
base: b/yo/master
Choose a base branch
from

Conversation

bellewether
Copy link

Wave 1: I believe all requirements were met...
Wave 2: My model tests are there, but my controller tests are lacking. I delayed submitting this project because even though I worked all weekend on it, I just couldn't get my Wave 1 requirements done in time. I didn't give myself enough opportunity to explore controller testing. Obviously, I want more practice with it, so I'm going to seek more opportunities to do controller testing in our next project (or come back to this one when I have time).
Wave 3: I utilized the Foundation grid to do some styling and I really like it as a tool. I definitely didn't give myself enough time to really explore the tool, so the styling is just hacked together. I couldn't get certain things, like my buttons, to display correctly. My buttons are a mess and I'm pretty sure I'm missing something when it comes to buttons and styling.

Overall, this is not the project I wanted to turn in, but this is what I've got right now.

…me controller for the homepage and an accompanying index view file
…zed spelling mistake in the controller and movie index view page
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants