diff options
| author | Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | 2016-09-27 12:42:42 +0000 | 
|---|---|---|
| committer | Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | 2016-09-27 12:42:42 +0000 | 
| commit | b0af83f9ff78773dbcd5435841bdac514d6e242f (patch) | |
| tree | cb6be81ffbbc649945e2f762140d109e258afc84 /lib | |
| parent | 166c6cd85c2be13bb2fa5d27f40d304ffd9c62ad (diff) | |
| parent | 0614cf19542a27ec232a69ab672242310d1b7733 (diff) | |
| download | gitlab-ce-b0af83f9ff78773dbcd5435841bdac514d6e242f.tar.gz | |
Merge branch 'dz-profile-organization' into 'master'
Add organization field to the user profile
## What does this MR do?
Add organization field to the user profile 
## Are there points in the code the reviewer needs to double check?
no
## Why was this MR needed?
So we can let users fill organization information separately
## Screenshots (if relevant)

## Does this MR meet the acceptance criteria?
- [x] [CHANGELOG](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CHANGELOG) entry added
- [x] ~~[Documentation created/updated](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/doc/development/doc_styleguide.md)~~
- [x] API support added
- Tests
  - [x] Added for this feature/bug
  - [x] All builds are passing
- [x] Conform by the [merge request performance guides](http://docs.gitlab.com/ce/development/merge_request_performance_guidelines.html)
- [x] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides)
- [x] Branch has no merge conflicts with `master` (if you do - rebase it please)
- [x] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits)
## What are the relevant issue numbers?
https://gitlab.com/gitlab-org/gitlab-ce/issues/21903
See merge request !6526
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/api/entities.rb | 2 | ||||
| -rw-r--r-- | lib/api/users.rb | 6 | 
2 files changed, 5 insertions, 3 deletions
| diff --git a/lib/api/entities.rb b/lib/api/entities.rb index 92a6f29adb0..0adc118ba27 100644 --- a/lib/api/entities.rb +++ b/lib/api/entities.rb @@ -15,7 +15,7 @@ module API      class User < UserBasic        expose :created_at        expose :is_admin?, as: :is_admin -      expose :bio, :location, :skype, :linkedin, :twitter, :website_url +      expose :bio, :location, :skype, :linkedin, :twitter, :website_url, :organization      end      class Identity < Grape::Entity diff --git a/lib/api/users.rb b/lib/api/users.rb index c440305ff0f..18c4cad09ae 100644 --- a/lib/api/users.rb +++ b/lib/api/users.rb @@ -60,6 +60,7 @@ module API        #   linkedin                          - Linkedin        #   twitter                           - Twitter account        #   website_url                       - Website url +      #   organization                      - Organization        #   projects_limit                    - Number of projects user can create        #   extern_uid                        - External authentication provider UID        #   provider                          - External provider @@ -74,7 +75,7 @@ module API        post do          authenticated_as_admin!          required_attributes! [:email, :password, :name, :username] -        attrs = attributes_for_keys [:email, :name, :password, :skype, :linkedin, :twitter, :projects_limit, :username, :bio, :location, :can_create_group, :admin, :confirm, :external] +        attrs = attributes_for_keys [:email, :name, :password, :skype, :linkedin, :twitter, :projects_limit, :username, :bio, :location, :can_create_group, :admin, :confirm, :external, :organization]          admin = attrs.delete(:admin)          confirm = !(attrs.delete(:confirm) =~ /(false|f|no|0)$/i)          user = User.build_user(attrs) @@ -111,6 +112,7 @@ module API        #   linkedin                          - Linkedin        #   twitter                           - Twitter account        #   website_url                       - Website url +      #   organization                      - Organization        #   projects_limit                    - Limit projects each user can create        #   bio                               - Bio        #   location                          - Location of the user @@ -122,7 +124,7 @@ module API        put ":id" do          authenticated_as_admin! -        attrs = attributes_for_keys [:email, :name, :password, :skype, :linkedin, :twitter, :website_url, :projects_limit, :username, :bio, :location, :can_create_group, :admin, :external] +        attrs = attributes_for_keys [:email, :name, :password, :skype, :linkedin, :twitter, :website_url, :projects_limit, :username, :bio, :location, :can_create_group, :admin, :external, :organization]          user = User.find(params[:id])          not_found!('User') unless user | 
