Skip to content

Add configuration management#34869

Open
user312222 wants to merge 2 commits into
RocketChat:developfrom
user312222:add-config-management
Open

Add configuration management#34869
user312222 wants to merge 2 commits into
RocketChat:developfrom
user312222:add-config-management

Conversation

@user312222
Copy link
Copy Markdown

feat: Add configuration management to improve the system

This Pull Request adds a new system to manage configurations. It stores all configuration data in one place. This makes it easy to set values.It also makes it easy to retrieve and change values. This helps avoid putting values directly into the code and makes sure the configuration works the same in different environments.

Changes:
Added a configurationManager.ts file to manage configuration data. Connected configuration management to important parts of the system. Added a logger.ts file for better tracking of configuration changes and index.ts,ConfigService.ts, configurationManager.ts added for managing and organizing configuration data in a central place

Notes:
Please check the configuration setup for different environments. Any suggestions to improve this are welcome.

@user312222 user312222 requested a review from a team as a code owner January 3, 2025 16:57
@dionisio-bot
Copy link
Copy Markdown
Contributor

dionisio-bot Bot commented Jan 3, 2025

Looks like this PR is not ready to merge, because of the following issues:

  • This PR is missing the 'stat: QA assured' label
  • This PR is missing the required milestone or project
  • This PR has an invalid title

Please fix the issues and try again

If you have any trouble, please check the PR guidelines

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Jan 3, 2025

🦋 Changeset detected

Latest commit: a19add0

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 35 packages
Name Type
@rocket.chat/meteor Major
@rocket.chat/core-typings Major
@rocket.chat/rest-typings Major
@rocket.chat/uikit-playground Patch
@rocket.chat/api-client Patch
@rocket.chat/apps Patch
@rocket.chat/core-services Patch
@rocket.chat/cron Patch
@rocket.chat/ddp-client Patch
@rocket.chat/freeswitch Patch
@rocket.chat/fuselage-ui-kit Major
@rocket.chat/gazzodown Major
@rocket.chat/livechat Patch
@rocket.chat/model-typings Patch
@rocket.chat/ui-contexts Major
@rocket.chat/account-service Patch
@rocket.chat/authorization-service Patch
@rocket.chat/ddp-streamer Patch
@rocket.chat/omnichannel-transcript Patch
@rocket.chat/presence-service Patch
@rocket.chat/queue-worker Patch
@rocket.chat/stream-hub-service Patch
@rocket.chat/license Patch
@rocket.chat/omnichannel-services Patch
@rocket.chat/pdf-worker Patch
@rocket.chat/presence Patch
rocketchat-services Patch
@rocket.chat/models Patch
@rocket.chat/network-broker Patch
@rocket.chat/ui-avatar Major
@rocket.chat/ui-client Major
@rocket.chat/ui-video-conf Major
@rocket.chat/ui-voip Major
@rocket.chat/web-ui-registration Major
@rocket.chat/instance-status Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants