Initial changes for removing ja_resource from donation_goal_controller

[paulsullivanjr]

What's in this PR?

Remove ja_resource from DonationGoalController

Closes #889
Progress on: #864

[paulsullivanjr]

I'd like to get some help with this if possible. I have 8 failing controller tests around canary. I'm not positive if I need to make these changes in policy.ex and what the right changes are. I still have failing tests if I leave policy.ex in its original state. Any feedback would be appreciated. Thanks!

[vishaldeepak]

Seems to me that you need to make modifications on CodeCorps.Policy.DonationGoal. In this file functions earlier expected a changeset but with the new changes it should expect a map of params. A good example would be in CodeCorps.Policy.ProjectSkill

[begedin]

@paulsullivanjr Hate to do this, but considering the amount of work @landongrindheim put into #1068, I had to merge his PR, since it allowed us the close the milestone in one fell swoop.

Your work here is still very much appreciated. However, since the task is done as part of #1068, I have to close this one.