models/team: Replace NewTeam::new() fn with Builder pattern

[Turbo87]

Having two i32 arguments in the new() fn was a bit dangerous and could easily lead to someone unintentionally swapping the values. This commit changes the NewTeam construction to use the builder pattern with named fns instead to avoid the potential confusion.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.13%. Comparing base (d8888c6) to head (4317a49).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #9990      +/-   ##
==========================================
- Coverage   89.13%   89.13%   -0.01%     
==========================================
  Files         294      294              
  Lines       30605    30593      -12     
==========================================
- Hits        27279    27268      -11     
+ Misses       3326     3325       -1     

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