Skip to content

Conversation

@beccaelenzil
Copy link
Contributor

@beccaelenzil beccaelenzil commented Sep 14, 2022

The requirements have been updated such that all new tasks should have completed_at = null, and the way to update the completed_at time is with mark_complete or mark_incomplete actions. Therefore, these tests that are removed in this PR are inconsistent with those requirements.

Asana task

Copy link
Contributor

@kelsey-steven-ada kelsey-steven-ada left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@beccaelenzil beccaelenzil merged commit b75f800 into master Sep 15, 2022
@beccaelenzil beccaelenzil deleted the be/wave-3-tests branch September 15, 2022 15:02
MBsea21 referenced this pull request in MBsea21/task-list-api Nov 8, 2024
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.

3 participants