BigW Consortium Gitlab

  1. 17 Nov, 2016 1 commit
    • issue and mergerequest slash command for mattermost · 9d8ca60d
      Z.J. van de Weg authored
      This commit includes a couple of thing:
      - A chatops controller
      - Mattermost::CommandService
      - Mattermost::Commands::(IssueService|MergeRequestService)
      
      The controller is the point where mattermost, and later slack will have
      to fire their payload to. This in turn will execute the CommandService.
      
      Thats where the authentication and authorization should happen. So far
      this is not yet implemented. This should happen in later commits.
      
      Per subcommand, in case of `/gitlab issue show 123` issue whould be the
      subcommand, there is a service to parse the data, and fetch the
      resource. The resource is passed back to the CommandService which
      structures the data.
  2. 12 Nov, 2016 1 commit
    • Remove unneeded nil checks · 9869c480
      Z.J. van de Weg authored
      The library we're using, default_value_for, is configured to interpret nil
      as the default value, being ENABLED. Given the property can't be nil
      this check is unneeded.
  3. 09 Nov, 2016 1 commit
  4. 17 Oct, 2016 1 commit
  5. 05 Oct, 2016 1 commit
    • Fix project deletion when feature visibility is set to private · fea80aa1
      Stan Hu authored
      Projects that are destroyed are put in the pending_delete state.
      The ProjectDestroyWorker checks whether the current user has
      access, but since the ProjectFeature class uses the default scope
      of the Project, it will not be able to find the right project.
      
      This was a regression in 8.12 that caused the following stack trace:
      
      ```
      NoMethodError: undefined method `team' for nil:NilClass
        from app/models/project_feature.rb:62:in `get_permission'
        from app/models/project_feature.rb:34:in `feature_available?'
        from app/models/project.rb:21:in `feature_available?'
        from app/policies/project_policy.rb:170:in `disabled_features!'
        from app/policies/project_policy.rb:29:in `rules'
        from app/policies/base_policy.rb:82:in `block in abilities'
        from app/policies/base_policy.rb:113:in `collect_rules'
        from app/policies/base_policy.rb:82:in `abilities'
        from app/policies/base_policy.rb:50:in `abilities'
        from app/models/ability.rb:64:in `uncached_allowed'
        from app/models/ability.rb:58:in `allowed'
        from app/models/ability.rb:49:in `allowed?'
        from app/services/base_service.rb:11:in `can?'
        from lib/gitlab/metrics/instrumentation.rb:155:in `block in can?'
        from lib/gitlab/metrics/method_call.rb:23:in `measure'
        from lib/gitlab/metrics/instrumentation.rb:155:in `can?'
        from app/services/projects/destroy_service.rb:18:in `execute'
      ```
      
      Closes #22948
  6. 21 Sep, 2016 2 commits
  7. 01 Sep, 2016 1 commit