BigW Consortium Gitlab

  1. 07 Dec, 2015 10 commits
    • Satisfy Rubocop · ff08ce9c
      Douwe Maan authored
    • Merge branch 'fix/award-emoji-conflict-in-notes' into 'master' · 359d9460
      Grzegorz Bizon authored
      Fix problems with award-emoji-only comment
      
      This fixes a conflict between note with only a single emoji in content
      and award-emojis mechanisms.
      
      Closes #3734 
      
      cc @vsizov
      
      See merge request !1936
    • Merge branch 'fix-global-milestones-error-500' into 'master' · 104df74f
      Douwe Maan authored
      Fix Error 500 when creating global milestones with Unicode characters
      
      Two issues:
      
      1. The constraints in the resources were incorrect. Here's what it was before:
      
      ```
      group_milestone  GET /groups/:group_id/milestones/:id(.:format)  groups/milestones#show {:id=>/[a-zA-Z.0-9_\-]+(?<!\.atom)/, :group_id=>/[a-zA-Z.0-9_\-]+(?<!\.atom)/}
      ```
      
      In this case, id is actually the title of the milestone, which can be anything at the moment.
      
      After:
      ```
      group_milestone  GET /groups/:group_id/milestones/:id(.:format)  groups/milestones#show {:id=>/[^\/]+/, :group_id=>/[a-zA-Z.0-9_\-]+(?<!\.atom)/}
      ```
      
      2. `parameterize` would strip all Unicode characters, leaving a blank string. Rails would report something like:
      
      ```
      ActionView::Template::Error (No route matches {:action=>"show", :controller=>"groups/milestones", :group_id=>#<Group id: 48, name: "ops-dev", path: "ops-dev", owner_id: nil, created_at: "2015-11-15 08:55:30", updated_at: "2015-12-02 06:23:26", type: "Group", description: "", avatar: "sha1.c71e73d51af1865c1bbbf6208e10044d46c9bb93.png", public: false>, :id=>"", :title=>"肯定不是中文的问题"} missing required keys: [:id]):
      ```
      
      This change uses the babosa library to create a better slug, which surprisingly
      isn't actually used by the global milestone controllers. Instead, they use the
      title passed as a query string for some reason.
      
      Closes https://github.com/gitlabhq/gitlabhq/issues/9881
      
      See merge request !1983
    • Merge branch 'style-warning-about-mentioning-many-people-in-a-comment' into 'master' · 4c353143
      Douwe Maan authored
      Style warning about mentioning many people in a comment
      
      Fixes #2885 
      
      * Create issue:
      
          Before:
      
          ![Screenshot_2015-12-03_15.23.44](/uploads/6da9b326e47dc5e8c127a581af99403c/Screenshot_2015-12-03_15.23.44.png)
      
          After:
      
          ![Screenshot_2015-12-03_15.21.29](/uploads/bbe328d30a32d642c280bf2ad6d2832c/Screenshot_2015-12-03_15.21.29.png)  
          &nbsp; 
      
      * New merge request:
      
          Before:
      
          ![Screenshot_2015-12-03_15.24.06](/uploads/79d6f8f52ba2753aaecf6b6bbd04f0d0/Screenshot_2015-12-03_15.24.06.png)
      
          After:
      
          ![Screenshot_2015-12-03_15.21.48](/uploads/f01ea8e7d058a9f8b803ca1ce4f533ad/Screenshot_2015-12-03_15.21.48.png)
          &nbsp;
      
      * New comment in a merge request:
      
          Before:
      
          ![Screenshot_2015-12-03_15.11.23](/uploads/5d958375812d21d41e00694186c7ef7a/Screenshot_2015-12-03_15.11.23.png)
      
          After:
      
          ![Screenshot_2015-12-03_15.09.08](/uploads/3d6bde6aad46c7d9e53b70a47eecf7c0/Screenshot_2015-12-03_15.09.08.png)
          &nbsp;
      
      * New diff comment:
      
          Before:
      
          ![Screenshot_2015-12-03_15.25.50](/uploads/08ac9c261c1aa9693706da714c121e15/Screenshot_2015-12-03_15.25.50.png)
      
          After:
      
          ![Screenshot_2015-12-03_15.46.20](/uploads/07a8b8708ad825eb8cda7e5e40116b81/Screenshot_2015-12-03_15.46.20.png)
      
      See merge request !1971
    • Move changelog item · 839a8b92
      Douwe Maan authored
    • Merge branch 'issue_1156' · 63687178
      Douwe Maan authored
    • Remove changelog entry issue number · f5ec1ebe
      Douwe Maan authored
  2. 06 Dec, 2015 2 commits
    • Merge branch 'fix-gitlab-workhorse-initd-check' into 'master' · 234f4bf2
      Stan Hu authored
      Check GitLab Workhorse status in init.d script when reporting all components are up and running
      
      Closes https://github.com/gitlabhq/gitlabhq/issues/9869
      
      See merge request !1969
    • Merge branch 'duplicate_notifications_fix' into 'master' · e2c57a41
      Stan Hu authored
      Fixed duplicated issue note email notifications.
      
      Fixes #2560 
      
      
      
      See issue for the details.
      
      Without `uniq` modified tests were failing with:
      ```
           Failure/Error: notification.new_note(note)
             (Notify (class)).note_issue_email(21, 1)
                 expected: 1 time with arguments: (21, 1)
                 received: 2 times with arguments: (21, 1)
           # /home/bak1an/.rvm/gems/ruby-2.1.6@gitlab/gems/sidekiq-3.3.0/lib/sidekiq/extensions/action_mailer.rb:17:in `public_send'
           # /home/bak1an/.rvm/gems/ruby-2.1.6@gitlab/gems/sidekiq-3.3.0/lib/sidekiq/extensions/action_mailer.rb:17:in `perform'
           # /home/bak1an/.rvm/gems/ruby-2.1.6@gitlab/gems/sidekiq-3.3.0/lib/sidekiq/testing.rb:74:in `block in raw_push'
           # /home/bak1an/.rvm/gems/ruby-2.1.6@gitlab/gems/sidekiq-3.3.0/lib/sidekiq/testing.rb:69:in `each'
           # /home/bak1an/.rvm/gems/ruby-2.1.6@gitlab/gems/sidekiq-3.3.0/lib/sidekiq/testing.rb:69:in `raw_push'
           # /home/bak1an/.rvm/gems/ruby-2.1.6@gitlab/gems/sidekiq-3.3.0/lib/sidekiq/client.rb:68:in `push'
           # /home/bak1an/.rvm/gems/ruby-2.1.6@gitlab/gems/sidekiq-3.3.0/lib/sidekiq/worker.rb:85:in `client_push'
           # /home/bak1an/.rvm/gems/ruby-2.1.6@gitlab/gems/sidekiq-3.3.0/lib/sidekiq/extensions/generic_proxy.rb:19:in `method_missing'
           # ./app/services/notification_service.rb:144:in `block in new_note'
           # ./app/services/notification_service.rb:143:in `each'
           # ./app/services/notification_service.rb:143:in `new_note'
           # ./spec/services/notification_service_spec.rb:63:in `block (5 levels) in <top (required)>'
      ```
      
      I have also added `once` to all `should_email` checks within `notification_service_spec.rb` since it's probably the correct behavior to notify users only once on the same event. Nothing else failed out of the box but we can keep these assertions for future.
      
      See merge request !1925
  3. 05 Dec, 2015 7 commits
  4. 04 Dec, 2015 21 commits