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

fix(install): Remove obsolete rrdtool configuration and sources #7195

Merged
merged 8 commits into from
Apr 5, 2019
Merged

fix(install): Remove obsolete rrdtool configuration and sources #7195

merged 8 commits into from
Apr 5, 2019

Conversation

lpinsivy
Copy link
Contributor

  • Remove unused PHP sources
  • Remove unused options in "Administration > Parameters > RRDTool" menu
  • Clean centreon.options table for update (no entries during installation)

sc979
sc979 reviewed Feb 15, 2019
View reviewed changes
Copy link
Contributor

@sc979 sc979 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't we remove translations texts from the pot files too ?

www/include/Administration/parameters/rrdtool/form.php Outdated Show resolved Hide resolved
@lpinsivy
Copy link
Contributor Author

@sc979 it will be removed automatically next time we will update translation.

This was referenced Mar 12, 2019
Closed
Closed
@lpinsivy lpinsivy force-pushed the remove-obsolete-rrdtool-configuration branch from 459d0b0 to b7a9b01 Compare April 2, 2019 12:53
@lpinsivy
Copy link
Contributor Author

lpinsivy commented Apr 2, 2019

Rebase branch originally from 18.10.x to master and merge conflicts

@lpinsivy lpinsivy merged commit a9b5949 into master Apr 5, 2019
@lpinsivy lpinsivy deleted the remove-obsolete-rrdtool-configuration branch April 5, 2019 07:15
@adr-mo adr-mo mentioned this pull request Apr 29, 2019
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@sc979 sc979 sc979 approved these changes

@callapa callapa callapa approved these changes

Assignees
No one assigned
Labels
area/install
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@lpinsivy @callapa @sc979