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

C22 - Phoenix - Charday N. #73

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

Conversation

chardayneal
Copy link

No description provided.

Complete initial setup of project, including migrations directory being
using for Task Model changes.

Create tasks endpoint for creating, reading, updating and deleting.

Enable few tests from Wave 01
Create helper function to ensure task is created with proper details.
Handle appropriately should we receive request with
invalid details for new task.
Implement sorting feature by adding sort query param that
sorts tasks by title, ascending and descending
Created validate_model and create_model helper
methods to DRY up code and create reusable functions

Created an endpoint that marks tasks complete and incomplete

Updated task model only to revert changes to resolve issue.
Updated the goal model attributes with 'title' and migrated changes

Create GET, POST, PUT, DELETE endpoints for Goal model
Completed partial testing for endpoints
Implemented some asserts to a few tests to ensure our api responds
appropriately to an invalid request and that exception are in fact
raised.
Tests invalid requests made by user and ensure they're handle
appropriately
Create a one-to-many relationship with Goal and Task models

Create endpoints that allow users to view tasks by goal and
add task ids to a specified goal
Refactor code to ensure consistency throughout app

Refactor Goal models methods to_dict() and from_dict()
Code used legacy code and updated with session.get() preferred method
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