Issue#3525: Goal class has unused field: user

[syamjir]

Hi sillsdev,

I have removed the user-related properties and imports from the Goal class and simplified the constructor and related code.

Please review the changes and provide any feedback.

Thank you!

Resolves #3525

---

This change is 

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 66.51%. Comparing base (fcd1a2b) to head (120969f).
Report is 45 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #3526      +/-   ##
==========================================
- Coverage   74.55%   66.51%   -8.05%     
==========================================
  Files         286      235      -51     
  Lines       11022     5844    -5178     
  Branches     1344      712     -632     
==========================================
- Hits         8218     3887    -4331     
+ Misses       2417     1729     -688     
+ Partials      387      228     -159     

Flag	Coverage Δ
backend	?
frontend	66.51% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[imnasnainaec]

Thanks for contributing @syamjir.

Please delete the 4 lines completely, rather than leaving extra empty lines in their places.

[syamjir]

@imnasnainaec ,Okay, I will delete those empty lines and submit the pull request shortly.

[imnasnainaec]

Reviewed 1 of 1 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @syamjir)