Clean up the ownerAccountID in the canModifyTask method

[tgolen]

cc @Krishna2323 this was another cleanup that I found

Problem

canModifyTask has a third argument for ownerAccountID but nothing is ever passing that argument. Instead, the canModifyTask is getting the data from getTaskOwnerAccountID(). This presents several problems:

1. The argument is useless
2. getTaskOnwerAccountID() is only doing return taskReport?.ownerAccountID; which seems like not really worth having a method for

Solution

• Remove the third argument from canModifyTask()
• Replace all calls to getTaskOnwerAccountID() with report?.ownerAccountID

[tgolen]

Weekly Update

• I created an E/App PR for this

Next Steps

• Have the PR reviewed and merged

ETA

• Done by tomorrow Apr. 23

[tgolen]

Weekly Update

• The task functionality was reverted from NewDot, which makes my PR obsolete
• My PR has been placed on HOLD for now

Next Steps

• @tgolen Once the task functionality has been moved back to NewDot, I will need to update my PR to be ready to review again

ETA

• Unknown

[melvin-bot]

This issue has not been updated in over 15 days. @tgolen eroding to Monthly issue.

P.S. Is everyone reading this sure this is really a near-term priority? Be brave: if you disagree, go ahead and close it out. If someone disagrees, they'll reopen it, and if they don't: one less thing to do!

[melvin-bot]

@tgolen, this Monthly task hasn't been acted upon in 6 weeks; closing.

If you disagree, feel encouraged to reopen it -- but pick your least important issue to close instead.