Skip to content
This repository has been archived by the owner on Jul 23, 2022. It is now read-only.

Shorten exercise 3 #607

Draft
wants to merge 2 commits into
base: na-jan-2021-updates
Choose a base branch
from
Draft

Conversation

sdstolworthy
Copy link
Contributor

No description provided.

@sdstolworthy sdstolworthy changed the title wording changes Shorten exercise 3 Jan 27, 2021

<kbd>📝 todolist/src/store/mutations.js</kbd>
```javascript
MARK_TODO_IMPORTANT(state, index) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This method must be added to the todolist repo before this PR can be merged.


<kbd>📝 todolist/src/store/actions.js</kbd>
```javascript
updateTodo({ commit, state }, { id, important }) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This method must be added to the todolist repo before this PR can be merged.


<kbd>📝 todolist/tests/unit/javascript/actions.spec.js</kbd>
```javascript
it("should call MARK_TODO_IMPORTANT", done => {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

All tests in this section must be added to todolist before this PR can be merged

@sdstolworthy sdstolworthy changed the base branch from master to na-jan-2021-updates January 28, 2021 15:27
@jacobsee
Copy link
Member

jacobsee commented Feb 2, 2021

Pulled into #606 so that I could add the tabbed interface at the same time. It should give you co-author credit on that...
Screen Shot 2021-02-01 at 7 08 56 PM

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Look at making exercise 3 (TDD) simpler / less prone to mistakes.
3 participants