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

enh(feature-switch): fully manage feature switch #10101

Merged
merged 4 commits into from
Sep 14, 2021

Conversation

adr-mo
Copy link
Contributor

@adr-mo adr-mo commented Sep 9, 2021

Description

This Pull Request intends to fully support the deprecated pages when using the parameter
It means that when enabled all redirections will lead to the deprecated pages (Monitoring -> Status -> Service & Host)
Impacted pages are:

  • Monitoring > EventLogs > Event Logs
  • Monitoring > Status Details > Services Grid (display: details & summary)
  • Monitoring > Status Details > Services by Hostgroup (display: details & summary)
  • Monitoring > Status Details > Services by Servicegroup (display: details & summary)
  • Monitoring > Status Details > Hostgroups Summary

The options has been also renamed from "Show deprecated pages" to "Use deprecated pages" and a help tooltip has been added to explain the purpose of this option.

The Pull Request also adds new entry to the result of the endpoint getting information about the current connected user.

image

Fixes # (issue)

Type of change

  • Patch fixing an issue (non-breaking change)
  • New functionality (non-breaking change)
  • Breaking change (patch or feature) that might cause side effects breaking part of the Software
  • Updating documentation (missing information, typo...)

Target serie

  • 20.04.x
  • 20.10.x
  • 21.04.x
  • 21.10.x (master)

How this pull request can be tested ?

See Jira ticket for this

Checklist

  • I have 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).

@adr-mo adr-mo requested review from a team, callapa and jeremyjaouen September 9, 2021 10:51
@adr-mo adr-mo force-pushed the MON-10645-feature-switch branch from 6352230 to 6bbda73 Compare September 9, 2021 12:29
@adr-mo adr-mo requested a review from centreon-bot September 9, 2021 12:30
jeremyjaouen
jeremyjaouen previously approved these changes Sep 10, 2021
callapa
callapa previously approved these changes Sep 10, 2021
@adr-mo adr-mo dismissed stale reviews from callapa and jeremyjaouen via d6d461e September 10, 2021 15:34
@sonarqube-decoration
Copy link

@adr-mo adr-mo requested a review from callapa September 13, 2021 19:01
@adr-mo adr-mo merged commit e250ba8 into develop Sep 14, 2021
@adr-mo adr-mo deleted the MON-10645-feature-switch branch September 14, 2021 08:55
adr-mo added a commit that referenced this pull request Sep 14, 2021
* enh(feature-switch): fully manage feature switch

* centreon-bot: remove coding style issues

* use correct entry name in documentation

* enh(resource): add new function to build meta service uri
adr-mo added a commit that referenced this pull request Sep 14, 2021
* enh(feature-switch): fully manage feature switch

* centreon-bot: remove coding style issues

* use correct entry name in documentation

* enh(resource): add new function to build meta service uri
adr-mo added a commit that referenced this pull request Oct 11, 2021
* enh(feature-switch): fully manage feature switch

* centreon-bot: remove coding style issues

* use correct entry name in documentation

* enh(resource): add new function to build meta service uri
adr-mo added a commit that referenced this pull request Oct 11, 2021
* enh(feature-switch): fully manage feature switch

* centreon-bot: remove coding style issues

* use correct entry name in documentation

* enh(resource): add new function to build meta service uri
adr-mo added a commit that referenced this pull request Oct 11, 2021
* enh(feature-switch): fully manage feature switch

* centreon-bot: remove coding style issues

* use correct entry name in documentation

* enh(resource): add new function to build meta service uri
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.

3 participants