Skip to content

Conversation

@Ribas160
Copy link
Contributor

Fixes #123

@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.21%. Comparing base (07bc3a7) to head (f9c8b03).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #124   +/-   ##
=======================================
  Coverage   98.21%   98.21%           
=======================================
  Files          68       68           
  Lines        3296     3296           
=======================================
  Hits         3237     3237           
  Misses         33       33           
  Partials       26       26           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@andrii-bodnar andrii-bodnar requested a review from Copilot October 29, 2025 15:23
@andrii-bodnar andrii-bodnar changed the title fix: User key replaced by Team key in GroupsTeam structure (#123) fix: User key replaced by Team key in GroupsTeam structure Oct 29, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR corrects a field naming issue in the GroupsTeam model by renaming the User field to Team, which better reflects the actual data structure representing a team entity rather than a user. All associated test cases have been updated to reflect this change.

  • Renamed User field to Team in the GroupsTeam struct
  • Updated all test mock JSON responses to use "team" instead of "user"
  • Updated all test assertions to reference .Team instead of .User

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
crowdin/model/teams.go Renamed User field to Team in GroupsTeam struct with corresponding JSON tag update
crowdin/teams_test.go Updated all test cases (List, Get, Edit) to use team in mock responses and Team in assertions

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Member

@andrii-bodnar andrii-bodnar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Ribas160 thank you!

@andrii-bodnar andrii-bodnar merged commit 44b7647 into crowdin:main Oct 29, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix GroupsTeam structure

3 participants