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

docs/server update configuration.md #43

Merged
merged 2 commits into from Dec 18, 2016
Merged

docs/server update configuration.md #43

merged 2 commits into from Dec 18, 2016

Conversation

daftaupe
Copy link
Contributor

Adds the missing settings found in config.js and some examples.

Adds the missing settings found in config.js and some examples.
[zenburn]: /img/screenshots/zenburn_css.png
[zenburn_thumb]: /img/screenshots/zenburn_thumbnail.png (Zenburn CSS)

## autoload
Copy link
Member

Choose a reason for hiding this comment

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

This is going away in thelounge/thelounge#779, mind getting rid of it in this PR?

Remove autoload directive to be coherent with #779
@daftaupe
Copy link
Contributor Author

Done, I've remove the autoload section.

Copy link
Member

@astorije astorije left a comment

Choose a reason for hiding this comment

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

These could/should be improved a lot, but in the meantime, it's great to be up-to-date!

Thanks for your effort, @daftaupe!


You can override this setting by starting The Lounge like this:
`lounge start --port 80`
`lounge start -- --port 80`
Copy link
Member

Choose a reason for hiding this comment

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

This is supposed to be fixed in v2.2.0 (not released yet), so mental note to double-check that after release and update doc accordingly. Ideally, we should not need this.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes I just wrote it in order to avoid people coming and opening tickets all the time about this.

@astorije astorije merged commit 358f7f0 into thelounge:master Dec 18, 2016
@daftaupe daftaupe deleted the patch-1 branch February 5, 2017 11:58
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.

None yet

3 participants