Skip to content

Updating config options as ENVs in README, updating a few other links#371

Open
rgarcia6520 wants to merge 1 commit intomattermost:masterfrom
rgarcia6520:readme-config-url-updates
Open

Updating config options as ENVs in README, updating a few other links#371
rgarcia6520 wants to merge 1 commit intomattermost:masterfrom
rgarcia6520:readme-config-url-updates

Conversation

@rgarcia6520
Copy link
Copy Markdown

Signed-off-by: garcia.ryan garcia.ryan@solute.us

Summary

Updating section of README for mattermost team edition to use config values with ENV vars rather than configJSON items per issue #238 . Also updating a few other broken links.

Ticket Link

Fixes #238

Signed-off-by: garcia.ryan <garcia.ryan@solute.us>
@mattermod
Copy link
Copy Markdown
Contributor

Hello @rgarcia6520,

Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

Per the Mattermost Contribution Guide, we need to add you to the list of approved contributors for the Mattermost project.

Please help complete the Mattermost contribution license agreement?
Once you have signed the CLA, please comment with /check-cla and confirm that the CLA check is green.

This is a standard procedure for many open source projects.

Please let us know if you have any questions.

We are very happy to have you join our growing community! If you're not yet a member, please consider joining our Contributors community channel to meet other contributors and discuss new opportunities with the core team.

@rgarcia6520
Copy link
Copy Markdown
Author

/check-cla

@mattermod
Copy link
Copy Markdown
Contributor

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!

@flickerfly
Copy link
Copy Markdown

Any reviewers about that could give this a quick look?

@EugenMayer
Copy link
Copy Markdown

EugenMayer commented Jul 9, 2025

3 years passed. And people have been asked several times to provide this very PR over and over again.

Well, just to not ask people to create PRs if you then end up ignoring those all together :)

Beside that, this not beeing merged causes so much confusion:

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.

configJSON is not working as per readme

4 participants