Allow task assignment from new task page

Open
#1,174 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
4/5
Estimated time
3-5 days
Newbie friendliness
35/100
Issue type
Feature
Clarity
Mostly clear
Activity status
Stale
Tech stack
javascript
Domain
frontend

Research direction

Start with component #1172 and its component tests, then inspect project/tasks/task/new.hbs and the new-task route/controller, especially the existing skill-assignment flow. Implement the assignment UI and save behavior described in the subtasks, update the component tests, and verify the sidebar styling and task save path.

Written by the indexing model from the issue text.

Description

blocked Skill: CSS Skill: Ember.js Skill: HTML Skill: JavaScript Skill: Testing

Problem

Users which are allowed to do so should be able to assign tasks from the new task page. We should use the component extracted in #1172 to achieve this, so this will be marked as blocked until we're ready for that.

Since this is for a new task, it should be implemented similarly to skill assignments. Instead of immediately creating a userTask record as soon as the user makes a dropdown selection, we should assign the selected user to a controller/route property. Once the user then saves the task, we should create and save the userTask record automatically.

This means that the component defined in #1172 might have to be modified to allow external assignment of the user, and tests would have to be updated to support this.

Subtasks

  • Include component from #1172 in project/tasks/task/new.hbs
  • [ ] Modify component so it allows external assignment of the assigned user. This should be relatively simple and be just a matter of overwriting the component property.
  • Update component tests so they cover this case
  • [ ] Handle onChange action correctly in route/controller. It should work similarly to how skill assignment works.
  • Modify task save action on route/controller to also create a userTask once saving is done
  • Fix any styling issues that might occur from using the component in the sidebar instead of the task card.
Dominant language
JavaScript
Stars
120
Forks
75
No merged PRs in 30d

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from code-corps/code-corps-ember

All issues in code-corps/code-corps-ember

Similar issues

More JavaScript issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.