Skip to content

Conversation

Bennyoak
Copy link

…ogo on about view

Same thing as PR #82 but not configuration.js

@daviddahl
Copy link
Member

The only thing I would add (havent built it yet) is a comment about app.kloak_version and app.build are properties in the un-checked-in www/js/configuration.js

Also, change the kloak_version property to "public_version"

@Bennyoak
Copy link
Author

Yeah that should be a README note in www/js/ maybe?

@daviddahl
Copy link
Member

@Bennyoak yeah, adding that to the README is a great idea

Copy link
Member

Choose a reason for hiding this comment

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

we should change app.kloak_version to app.app_version as well as add a section to the README about the existence of these required properties that should be in configuration.js
perhaps the best thing to do is add a section for "Configuration" in the readme where you display a sample configuration.js that has comments for each line?

Copy link
Member

Choose a reason for hiding this comment

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

Other than that, I can merge this PR

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.

2 participants