Skip to content

Add skills to tasks #1045

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

Merged
merged 2 commits into from
Feb 15, 2017
Merged

Add skills to tasks #1045

merged 2 commits into from
Feb 15, 2017

Conversation

begedin
Copy link
Contributor

@begedin begedin commented Feb 13, 2017

What's in this PR?

Adds a task skill UI to the task sidebar on the task page. Allows assigning of task skills via a search input as well as removing assinged task skills.

Also adds a list of skills on the task card. For this purpose, I rewrote project-card-skills into related-skills, since behavior is near-identical and only styling is slightly different.

TODO:

References

Progress on #1042
Requires code-corps/code-corps-api#696

@begedin begedin changed the title 1042 add skills to tasks Add skills to tasks Feb 13, 2017
@begedin begedin mentioned this pull request Feb 13, 2017
4 tasks
@begedin begedin added this to the Assign skills to tasks milestone Feb 13, 2017
@begedin
Copy link
Contributor Author

begedin commented Feb 13, 2017

@joshsmith I went ahead and wrote a bunch of issues to split the TODO's into that. I think this is in a mergeable state as is, so we can focus on smaller issues/PRs individually (and in parallel)

@begedin begedin force-pushed the 1042-add-skills-to-tasks branch 3 times, most recently from 68e3c3e to e79df72 Compare February 14, 2017 10:45
@begedin begedin requested a review from joshsmith February 14, 2017 12:06
@joshsmith joshsmith self-assigned this Feb 14, 2017
@begedin begedin force-pushed the 1042-add-skills-to-tasks branch from e79df72 to 8cfdcb3 Compare February 15, 2017 09:39
@joshsmith joshsmith merged commit 0cc50ce into develop Feb 15, 2017
@joshsmith joshsmith deleted the 1042-add-skills-to-tasks branch February 15, 2017 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants