Skip to content
This repository has been archived by the owner on Dec 13, 2022. It is now read-only.

fix(acl) exclude hosts excluded from hg #7604

Merged
merged 1 commit into from
Jun 18, 2019
Merged

Conversation

loiclau
Copy link
Contributor

@loiclau loiclau commented Jun 12, 2019

Pull Request Template

Type of change

  • Patch fixing an issue (non-breaking change)

Target serie

  • 18.10.x
  • 19.04.x
  • 19.10.x (master)

How this pull request can be tested ?

Define an ACL resource like
all hosts
all hostgroups
exclude some hosts

Given a user with ACL limiting resource access
When I go to Reporting > Dashboard > By hostgroups
I can see only my hosts

Community contributors & Centreon team

  • I followed the coding style guidelines provided by Centreon
  • I have commented my code, especially new classes, functions or any legacy code modified. (docblock)
  • I have commented my code, especially hard-to-understand areas of the PR.
  • I have made corresponding changes to the documentation.
  • I have rebased my development branch on the base branch (master, maintenance).

Centreon team only

  • I have made sure that the unit tests related to the story are successful.
  • I have made sure that unit tests cover 80% of the code written for the story.
  • I have made sure that acceptance tests related to the story are successful (local and CI)

@loiclau loiclau merged commit 5a01a79 into 19.04.x Jun 18, 2019
loiclau added a commit that referenced this pull request Jun 18, 2019
loiclau added a commit that referenced this pull request Jun 18, 2019
@loiclau loiclau deleted the MON-3853-host-acl-in-hg branch June 18, 2019 14:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants