Skip to content

Conversation

@dayures
Copy link
Contributor

@dayures dayures commented May 6, 2025

No description provided.

@dayures
Copy link
Contributor Author

dayures commented Jul 4, 2025

@jason-p-pickering could you review this?

@jason-p-pickering
Copy link
Contributor

will defer to @luciano-fiandesio

Copy link
Contributor

@bobjolliffe bobjolliffe left a comment

Choose a reason for hiding this comment

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

Great to see these keys coming in to the documentation. I would recommend commenting out the actual settings in this case. Sometimes people will take the reference conf file and modify it (I sometimes do that). In that case they might miss to comment these out. By default there should be no analytics.database settings. Unless you do indeed have another separate postgresql database for this

@dayures
Copy link
Contributor Author

dayures commented Sep 19, 2025

Thanks for the feedback @bobjolliffe . I am happy to contribute in the most useful way. BTW, there is a big note before the file content saying "You should not attempt to use this configuration file directly, rather use it as a reference for the available configuration options. Many of the properties are optional."

How should be added the "analytics.database" parameters? Adding them commented out ?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants