Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update note on requirements for Huginn statistics #643

Merged
merged 1 commit into from
Dec 25, 2022

Conversation

cameronwp
Copy link
Contributor

@cameronwp cameronwp commented Dec 24, 2022

Description

Update docs on requirements for Huginn's endpoints. It's required that PUBLIC=1 for odin to pull stats from the server. See this discussion

I didn't test locally, but no code changes were made.

Contributions

  • I added docs on requiring PUBLIC=1 to get stats from Huginn because it is an undocumented requirement.

Checklist

  • I added one or multiple labels which best describes this PR.
  • I have tested the changes locally.
  • This PR has a reviewer on it.
  • I have validated my changes in a docker container and on Ubuntu. (Only needed for Odin or Docker Changes)

@cameronwp
Copy link
Contributor Author

To the maintainers: apologies. For the life of me, I can't figure out how to add a label to the PR.

@mbround18 mbround18 added good first issue Good for newcomers documentation Changes only affect the documentation patch Increment the patch version when merged labels Dec 24, 2022
@mbround18 mbround18 merged commit b7cc009 into mbround18:main Dec 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Changes only affect the documentation good first issue Good for newcomers patch Increment the patch version when merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants