diff --git a/modules/structs/admin_user.go b/modules/structs/admin_user.go index ee1738579ab..5da4e9608be 100644 --- a/modules/structs/admin_user.go +++ b/modules/structs/admin_user.go @@ -32,8 +32,9 @@ type EditUserOption struct { FullName *string `json:"full_name" binding:"MaxSize(100)"` Password string `json:"password" binding:"MaxSize(255)"` MustChangePassword *bool `json:"must_change_password"` - Website *string `json:"website" binding:"MaxSize(50)"` + Website *string `json:"website" binding:"OmitEmpty;ValidUrl;MaxSize(255)"` Location *string `json:"location" binding:"MaxSize(50)"` + Description *string `json:"description" binding:"MaxSize(255)"` Active *bool `json:"active"` Admin *bool `json:"admin"` AllowGitHook *bool `json:"allow_git_hook"` diff --git a/modules/structs/org.go b/modules/structs/org.go index 4b79a4e70a0..483f5044a89 100644 --- a/modules/structs/org.go +++ b/modules/structs/org.go @@ -22,9 +22,9 @@ type CreateOrgOption struct { // required: true UserName string `json:"username" binding:"Required"` FullName string `json:"full_name"` - Description string `json:"description"` - Website string `json:"website"` - Location string `json:"location"` + Description string `json:"description" binding:"MaxSize(255)"` + Website string `json:"website" binding:"ValidUrl;MaxSize(255)"` + Location string `json:"location" binding:"MaxSize(50)"` // possible values are `public` (default), `limited` or `private` // enum: public,limited,private Visibility string `json:"visibility" binding:"In(,public,limited,private)"` @@ -34,9 +34,9 @@ type CreateOrgOption struct { // EditOrgOption options for editing an organization type EditOrgOption struct { FullName string `json:"full_name"` - Description string `json:"description"` - Website string `json:"website"` - Location string `json:"location"` + Description string `json:"description" binding:"MaxSize(255)"` + Website string `json:"website" binding:"ValidUrl;MaxSize(255)"` + Location string `json:"location" binding:"MaxSize(50)"` // possible values are `public`, `limited` or `private` // enum: public,limited,private Visibility string `json:"visibility" binding:"In(,public,limited,private)"` diff --git a/modules/structs/user.go b/modules/structs/user.go index efd246e566a..b5d5b9298ef 100644 --- a/modules/structs/user.go +++ b/modules/structs/user.go @@ -37,8 +37,8 @@ type User struct { Location string `json:"location"` // the user's website Website string `json:"website"` - // the user's biography - Description string `json:"bio"` + // the user's description + Description string `json:"description"` } // MarshalJSON implements the json.Marshaler interface for User, adding field(s) for backward compatibility diff --git a/routers/api/v1/admin/user.go b/routers/api/v1/admin/user.go index 5a74c6ccd50..2d4a3815f4c 100644 --- a/routers/api/v1/admin/user.go +++ b/routers/api/v1/admin/user.go @@ -203,6 +203,9 @@ func EditUser(ctx *context.APIContext) { if form.Location != nil { u.Location = *form.Location } + if form.Description != nil { + u.Description = *form.Description + } if form.Active != nil { u.IsActive = *form.Active } diff --git a/templates/swagger/v1_json.tmpl b/templates/swagger/v1_json.tmpl index 6d690ba275a..8255ed6e6ea 100644 --- a/templates/swagger/v1_json.tmpl +++ b/templates/swagger/v1_json.tmpl @@ -13800,6 +13800,10 @@ "type": "boolean", "x-go-name": "AllowImportLocal" }, + "description": { + "type": "string", + "x-go-name": "Description" + }, "email": { "type": "string", "format": "email", @@ -16251,16 +16255,16 @@ "type": "string", "x-go-name": "AvatarURL" }, - "bio": { - "description": "the user's biography", - "type": "string", - "x-go-name": "Description" - }, "created": { "type": "string", "format": "date-time", "x-go-name": "Created" }, + "description": { + "description": "the user's description", + "type": "string", + "x-go-name": "Description" + }, "email": { "type": "string", "format": "email",