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's API Muncher #71

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

Conversation

bellewether
Copy link

Baseline requirements met.

I used a gem for pagination, but after talking with other classmates this weekend, I realized that we weren't meant to use the gem if we didn't know how to paginate without it. I don't know how to do that, but I plan to talk to someone this week who can walk me through it.

I don't know what memoization is, and I didn't realize that it was something we were meant to implement until Friday. Hopefully, it'll become clearer this week.

I chose to spend more time improving my use of semantic HTML, Foundation and general CSS styling than I have on past projects. The look and feel of my site is still pretty simplistic, but I'm glad I focused on playing around more with Foundation and CSS this project. Still, I think there is room for improvement in DRYing up the CSS.

Testing is sparse. My tests rely too much on the examples from class, and I'm having trouble going beyond that.

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