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 74.59%. Comparing base (0947624) to head (c2d01a9).

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #3526      +/-   ##
==========================================
+ Coverage   74.55%   74.59%   +0.04%     
==========================================
  Files         286      286              
  Lines       11022    11022              
  Branches     1344     1344              
==========================================
+ Hits         8217     8222       +5     
+ Misses       2418     2414       -4     
+ Partials      387      386       -1     

Flag	Coverage Δ
backend	83.71% <ø> (+0.09%)	⬆️
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.