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

M. Johnson MediaRanker B/mbj/master #83

Open
wants to merge 22 commits into
base: b/mbj/master
Choose a base branch
from

Conversation

mjohnson3038
Copy link

I think this has been by far the hardest project for me by far. There were a lot of things that I spent a LONG time working through and still had no success. My biggest challenges were working with the controller test for rank/vote. I left some of my thoughts in as comments, but for some reason, it would not go through and work. I also struggled with the other tests because I think I didn't fully understand how to use the fixture and its relationship with the test database. May we please talk about this some more. I spent too much time working on the controller tests and I still feel that there are many things I wasn't able to consider within them. Although I created the CSS Framework early on, I saved the CSS styling for last and didn't realize how different this would be from regular CSS styling. I was able to make some changes for the files in the application.css but for some reason, it would not allow me to put customized css. For example, I tried to add a class = "button" and then I was going to style all the buttons, but for some reason this was not working. This was in a section were there was also another class, but as far as I could tell, this should not have impacted it. I ultimately did not budget enough time for these final touch ups after spending too much time trying to work through the controller tests. I think it might be helpful to go over the successful completed project to help see examples of things I struggled with.

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.

2 participants