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

Assignment Submission #2

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Assignment Submission #2

wants to merge 8 commits into from

Conversation

gamov
Copy link

@gamov gamov commented Jun 23, 2016

I've left comments generated by Rails throughout the app. I find those useful to develop the app further.

gamov added 8 commits June 5, 2016 00:26
New rails app
Setting up environment (gems)
First shot at the models
- youtube data retrieval
- video object interface
- videos UX (add/view/remove)

Some refactoring and clean up.
Change requirement to Ruby 2.3.1

Remark: we don't test the UI yet, it might still change drastically.
- boards functionality and UX
- misc improvements

App is now feature complete. We can start refining it and writing integration tests.
…d current migration because SQLite doesn't like adding not null column to an existing table...

- Removed my Youtube API key (and revoked it...) and set it via ENV variables (see secrets.yml)
- Refined UI/X
- misc bug fixes and improvements
- storing current board in session for more consistant navigation (and cleaner code)
- normalized css names to rails style
- some refactoring
- handles ajax errors
- sprinkled some css (can't help it!)

Left with integration testing
- bug fixes, misc. improvements
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