Assign task to multiple people

[james1293]

Hello all,
I just started using Django todo, and I love the simplicity of it. I recently hacked together a fork (locally, not on GitHub) that allows a task to be assigned to multiple users. It's not perfect yet, but would this be something worth sharing? If so, would it be the sort of thing that would stay as a fork, or could it be incorporated into the main project repo?
(I haven't contributed much on GitHub, so that's why I'm asking for etiquette advice.)
Thanks!

[shacker]

Hi James - this is a great idea. It should be as simple as converting the ForeignKey from Task to User to a ManyToMany field, and modifying the UI to allow multi-assignment. It would also have to ensure that emails are sent to all of the right parties.

The general process for contributing to open source on github is this:

1. Fork the project (you already have)
2. Make your changes in a branch and push the branch to your fork's repo
3. Make a pull request for that branch, targeted at this repo
4. We'll go through the review process - I'll suggest changes, you'll push additional commits
5. I'll merge it and it will be in!

Thanks.

[GoldenBookCover]

I tried simply converting the ForeignKey from Task to User to a ManyToMany field, it worked almost correctly, except the task_detail view, where i couldn't update the "Assigned to:" column. However it worked fine when creating a new task, or editing the task from the admin panel. Any suggestion?

[james1293]

Two things -
You have to add form.save_m2m()
For the list display, I did .join(", ")

[shacker]

@MonstreCharmant You would need to replace the Assigned To dropdown with some sort of multiple-choice selection widget. Or did you do that already and I'm misunderstanding the question?

[GoldenBookCover]

Thank you guys, I just added form.save_m2m(), it worked fine. The code looks like this now, I'm not sure if there is any impact to my data?

# from task_detail.py --> def task_detail() --> # Save task edits
if form.is_valid():
            item = form.save(commit=False)
            item.note = bleach.clean(form.cleaned_data["note"], strip=True)
            item.save()
            form.save_m2m()
            messages.success(request, "The task has been edited.")
            return redirect(
                "todo:list_detail",
                list_id=task.task_list.id,
                list_slug=task.task_list.slug,
            )

[james1293]

@MonstreCharmant I added form.save_m2m() in the same place, and it seems to be working. Would you like to share your fork, or do you want me to share mine? Mine is messy (I commented tests that didn't work any more, and I disabled the email notifications) but I could share it as-is anyway...

[shacker]

Still hoping that one of you can share your changes back to the mother project here! Pull request please?

[GoldenBookCover]

Yes, will do that. Working on some jobs now, but I'll share my changes once I got some time.

[f4i5i]

Yes, will do that. Working on some jobs now, but I'll share my changes once I got some time.

can you share your changes

[james1293]

I'm sorry that I never did share mine. I'm no longer working at the job where I was working on this, so I don't think that I would be able to access the code easily. If you decide to reimplement it, I wish you the best of luck.

[f4i5i]

can you guide me how you done the task few tips

[james1293]

It has been 2 years, so I don't remember a lot of the details. It was something like this:

• Remove the assigned_to field here.
• Add a field called assigned_users (or similar). It would look like this: assigned_users = models.ManyToManyField(...).
• Run a database migration.
• Search the code for occurrences of assigned_to. Change them appropriately.
  • In particular, when saving a task, you'll need to run form.save_m2m(), as described earlier in this thread.

Feel free to ask follow-up questions. 🙂

[f4i5i]

okay what for selection of users from template

[james1293]

I think I started by making sure it worked using the default multi-select interface. Then, once I was sure that that worked, I figured out how to use the pretty widget from the Django admin. There's some advice here (I recommend reading the comments on that answer).

Does that make sense?