Skip to content

Crow - Riley Drellishak #14

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 70 commits into
base: main
Choose a base branch
from
Open

Crow - Riley Drellishak #14

wants to merge 70 commits into from

Conversation

@rileydrellishak
Copy link

@rileydrellishak rileydrellishak commented Nov 6, 2025

Task List API Project!

@rileydrellishak
Sets up task list API project, includes migrations
1cab6bc
@rileydrellishak
Completes attributes for Task model
d3793c9
@rileydrellishak
First migration after creating Task class
d22d79e
@rileydrellishak
Implements from_dict class method for Task, but need to figure out ho…
3615b4d 
…w to connect completed_at to is_complete
@rileydrellishak
implements class method from_dict and instance method to_dict, remove…
e0c8a6f 
…s decorators for these tests and passes
@rileydrellishak
registers task bp in app, creates skeleton task routes for get, post,…
9cd3719 
… put, and delete
@rileydrellishak
creates route_utilities to make a validate_model helper function, imp…
d19585b 
…lements get all tasks route, tests passing
@rileydrellishak
Implements routes for get all tasks and get one task by id, handles i…
0c92b29 
…nvalid and non existent id
@rileydrellishak
Implements create task route, but the from_dict function is a bit fun…
ebc03a5 
…ky. all the tests are passing
@rileydrellishak
implements update task routes, passes all tests for put route
1809340
@rileydrellishak
Implements delete task route, all tests pass with appropriate error m…
dc216e3 
…essage asserts implemented.
@rileydrellishak
Deletes extra whitespace lines. Finished wave 1 but the from_dict cla…
81d74c8 
…ss method for tasks is a bit weird (the if statements) - will revisit.
@rileydrellishak
adds a message to the end of route_utilities about dir() and __dir__,…
f0bd81b 
… which could help with generalizing the route helper function to create an instance of a class from a dictionary so I don't have to define to_dict and from_dict for both models.
@rileydrellishak
Merge pull request #1 from rileydrellishak/wave-1
47af6c2 
Wave 1 (initial merge)
@rileydrellishak
Implements sorting for get all tasks route
5d3fc52
@rileydrellishak
Merge pull request #2 from rileydrellishak/wave-2
d7e367b 
Wave 2
@rileydrellishak
Implements mark_complete method, passes wave 3 tests with it.
b696355
@rileydrellishak
Implements mark_incompleete and passes tests. I feel like I can combi…
0d1ad37 
…ne these routes with a <how_to_mark> endpoint?
@rileydrellishak
Combined mark complete and incomplete in same route
61a6d6a
@rileydrellishak
Merge pull request #3 from rileydrellishak/wave-3
01981b3 
Wave 3
@rileydrellishak
Separated mark incomplete and mark complete back into two separate en…
6e30cfc 
…dpoints
@rileydrellishak
Fixed complete/incomplete (I had them swapped), successfully posts to…
1007818 
… the slack channel!!! Yay!!!
@rileydrellishak
Merge pull request #4 from rileydrellishak/wave-4
be1c331 
Wave 4
@rileydrellishak
Implements Goal model, to_dict, from_dict methods for Goal. Will crea…
cb4f79e 
…te a model_utilities to generalize the to and from dict methods
@rileydrellishak
Registers goal model bp, implements get one goal by id, implements mi…
30cda81 
…gration for new table in db for goals
@rileydrellishak
Creates model utilities file, updates goal routes and tests for wave 5
ddb8362
@rileydrellishak
Merge pull request #5 from rileydrellishak/wave-5
62f8a66 
Wave 5
@rileydrellishak
Establishes one to many relationship for goal to tasks, migration inc…
7411f0a 
…luded
@rileydrellishak
implements send list of tasks id to assign to goal
16f9531
@rileydrellishak
implements get tasks from a goal id
740983a
@rileydrellishak
Merge pull request #8 from rileydrellishak/optional-list-comprehensions
e3d612c 
Optional list comprehensions
@rileydrellishak
Implements wave 7 assert statements and cleans up comments
16f1c1c
@rileydrellishak
Merge pull request #9 from rileydrellishak/test-wave-07
b26db88 
Implements wave 7 assert statements and cleans up comments
@rileydrellishak
Updates wave 5 test to match updated patch method
89c1804
@rileydrellishak
Adds tasks attribute to Goal, adds goal_id and goal attributes to Task
ab55763
@rileydrellishak
Registers goal routes bp
6adb2dd
@rileydrellishak
create_model route helper, goal routes for create, get all, and delet…
5c7c412 
…e one by id
@rileydrellishak
post task ids to goal route implemented, passes tests
067a983
@rileydrellishak
updates wave 6 methods because I thought I really messed them up
d9739cb
@rileydrellishak
Merge branch 'main' into redo-wave-6
f58bef1
@rileydrellishak
Merge pull request #10 from rileydrellishak/redo-wave-6
3746f1b 
Redo wave 6
@rileydrellishak
deleted a double line
468abc7
@rileydrellishak
writes the patch method to update goal title
2f9fe6a
@rileydrellishak
generalizes patch to include more than one attribute to update
61aa819
@rileydrellishak
Merge pull request #11 from rileydrellishak/redo-wave-5
0e90274 
Redo wave 5
@rileydrellishak
fixes reassigning attribute value when making patch request
478dedc
@rileydrellishak
Merge branch 'main' into redo-wave-5
f4221e2
@rileydrellishak
minor fix to for attr, value in request_body.items() goal patch method
cf97276
@rileydrellishak
implements get_models_with_filters that also sorts the query by title…
c15d21f 
…. works for both goal and task and passes all tests after refactoring.
@rileydrellishak
adds three goals fixture for optional feature testing
23c44c8
@rileydrellishak
expands upon get_models_with_filters to include logic to sort by any …
b0bfe9c 
…column in either direction
@rileydrellishak
writes tests to test filtering and sorting method
4daac79
@rileydrellishak
finishes tests for query params to determine sort direction and by wh…
c882bda 
…ich column to sort by
@rileydrellishak
Merge pull request #12 from rileydrellishak/query-params-helper
8570627 
Query params helper
@rileydrellishak
added more tests to test_query_params_refactor for 100% test coverage
c5367e6
@rileydrellishak
final run through of methods to make sure they all worked, updated th…
15883a2 
…e patch goal to actually be a put like the readme says
@rileydrellishak
updated tests to show that the put and patch methods persisted data
5abd3a3
@rileydrellishak
moved slack post message to its own helper function in route utilities
e196203
@rileydrellishak
adds if not slack token line to send_slack_message helper function
05a1453
@rileydrellishak
another update to the slack token get
947dfd3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@rileydrellishak