Skip to content

OAuth TaskList - Yasmin #66

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

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

Conversation

bellewether
Copy link

I think that I met the requirements... I wasn't sure what changes to make to the task controller's new action since a new task is saved with the user's id in the create action, so the new action is still unchanged. I wanted to write controller tests, especially for the sessions controller, but I didn't have time to. It's just model tests for this project.

…pe from boolean to datetime, and the other to change the name from completion to completed_at
…on that will render as a new view for the user
…also changed the completed_at field in the form for easier entry
…ound setting up relationships between tasks and users as well as tinkering with authentication routes
… the schema) and added validations to the task model
…e login method to be required before every action
… added the route and the action for login, also adjusted controller filter
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