Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[API] orgEditTeam make Fields optional #9556

Merged
merged 22 commits into from
Jan 9, 2020
Merged

[API] orgEditTeam make Fields optional #9556

Changes from 1 commit
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
b49e4a8
API: orgEditTeam make Fields optional
6543 Dec 31, 2019
ecc15a5
add TestCase
6543 Jan 2, 2020
0c76d5c
Update integrations/api_team_test.go
6543 Jan 2, 2020
c5ed8bf
Merge branch 'master' into api-orgEditTeam_make-Name-optional
6543 Jan 2, 2020
a9a0f9b
suggestions from lafriks
6543 Jan 2, 2020
89ef6aa
Merge branch 'master' into api-orgEditTeam_make-Name-optional
6543 Jan 2, 2020
322d657
change ...
6543 Jan 4, 2020
bae6724
use Where not ID to get mssql
6543 Jan 4, 2020
ad12e31
add return and code format
6543 Jan 4, 2020
ccb0f9f
Merge branch 'master' into api-orgEditTeam_make-Name-optional
6543 Jan 4, 2020
f52f736
fix test
6543 Jan 4, 2020
02a87c9
fix test ... null pointer exept
6543 Jan 4, 2020
c263119
update specific colums
6543 Jan 4, 2020
6d36861
only specific colums too
6543 Jan 4, 2020
2dadeea
Merge branch 'master' into api-orgEditTeam_make-Name-optional
6543 Jan 4, 2020
3b41462
Merge branch 'master' into api-orgEditTeam_make-Name-optional
6543 Jan 6, 2020
60096e4
Merge branch 'master' into api-orgEditTeam_make-Name-optional
6543 Jan 6, 2020
23fef55
Merge branch 'master' into api-orgEditTeam_make-Name-optional
6543 Jan 7, 2020
6078b3c
Merge branch 'master' into api-orgEditTeam_make-Name-optional
lunny Jan 8, 2020
67d60ab
Merge branch 'master' into api-orgEditTeam_make-Name-optional
6543 Jan 8, 2020
1bd9e32
Merge branch 'master' into api-orgEditTeam_make-Name-optional
6543 Jan 9, 2020
cb9b3a9
Merge branch 'master' into api-orgEditTeam_make-Name-optional
6543 Jan 9, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
Prev Previous commit
Next Next commit
use Where not ID to get mssql
  • Loading branch information
@6543
6543 committed Jan 4, 2020
commit bae6724216f5b3bf367910e15a9b3d0b4197d495
2 changes: 1 addition & 1 deletion models/org_team.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -590,7 +590,7 @@ func UpdateTeam(t *Team, authChanged bool, includeAllChanged bool) (err error) {
return ErrTeamAlreadyExist{t.OrgID, t.LowerName}
}

if _, err = sess.ID(t.ID).AllCols().Update(t); err != nil {
if _, err = sess.Where("team.id = ?", t.ID).AllCols().Update(t); err != nil {
return fmt.Errorf("update: %v", err)
}

Expand Down