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

Revised TaskListRails project - Rachel Pavilanis #71

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

Conversation

rpavilanis
Copy link

No description provided.

rpavilanis and others added 30 commits September 28, 2016 13:49
…idual task lists. created new layout so that current user list page has a link to add a new person.
… as is my ability to mark things as complete. I am able to delete tasks, however, my redirect_to: index isn't working in my destroy controller method, or really anywhere. I am able to go into edit something, but then the edit form is blank and even if I change nothing and hit edit, the form then becomes blank. I really want to get this working with having the different layouts and people with their various tasks, so I will probably keep working on this.
…t working for new task now that I have users.
rpavilanis and others added 23 commits October 5, 2016 22:46
…res to work by adding a parameter in when I created a new task and adjusting my routes file to accept a person_id when creating a new task. Feeling SO proud of myself right now. LOL.
…inutes. But I got it working - a link just broke in my layout for tasks because I changed new to accept a parameter. It all seems to be working now, this will be my final submission for this, I think.
added session controller and user model, started to modify files for …
added model tests for user model
changed config.assets.compile to true
…innier based on things I have learned since the initial project - added find_user, find_task, and task_params private methods to task controller.
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