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

Starting tasks support #78

Merged
merged 4 commits into from
Mar 9, 2016
Merged

Starting tasks support #78

merged 4 commits into from
Mar 9, 2016

Conversation

enxebre
Copy link
Contributor

@enxebre enxebre commented Mar 8, 2016

This introduce very basic support for tasks
Follow ups:
#76
#77
#75

Table
screen shot 2016-03-08 at 17 41 13
Visualizer
screen shot 2016-03-08 at 17 40 50

@wallies
Copy link
Contributor

wallies commented Mar 8, 2016

these codeclimate build failures are fixed by #79

import Circle from '../../components/Circle';
import random from 'lodash/number/random';
import range from 'lodash/utility/range';
import List from 'material-ui/lib/lists/list';
Copy link
Member

Choose a reason for hiding this comment

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

v minor but lines like these (theres a few other places) can be condensed to import { List, ListItem } from 'material-ui/lib/lists'

@tayzlor
Copy link
Member

tayzlor commented Mar 9, 2016

🎉

@enxebre enxebre force-pushed the starting-tasks-support branch 2 times, most recently from 6303d46 to 709405d Compare March 9, 2016 16:00
enxebre added a commit that referenced this pull request Mar 9, 2016
@enxebre enxebre merged commit 6870c1b into master Mar 9, 2016
@enxebre enxebre deleted the starting-tasks-support branch March 16, 2016 23:56
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