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

Web interface for the log module #325

Closed
wants to merge 5 commits into from
Closed

Conversation

pepijndevos
Copy link

I thought it'd be cool to be able to view logs in the web interface.

It took me forever to write, since it's my first piece of C++.

@urbels
Copy link

urbels commented Apr 20, 2013

Niceone. ;)

@pepijndevos
Copy link
Author

One thing that this code does not handle, and is beyond my C++ skills: If I set the path to be $USER/$NETWORK/$CHANNEL-%Y%M%D.log or something like that, my logic would break. It asumes one dir containing all log files, not some weird nested structure.

@DarthGandalf
Copy link
Member

Pages are split regardless to line endings?

@pepijndevos
Copy link
Author

Yes.

@DarthGandalf
Copy link
Member

Are you ok with #311?

@pepijndevos
Copy link
Author

Yes.

@pepijndevos
Copy link
Author

@DarthGandalf I saw you comments on IRC. I'm not sure pagination works correctly. I tested it on some lorem ipsum. My logs are never over 1MB for one channel for one day.

I'm definitely going to work some more on this module. It get quite unwieldy after a while. I should write proper line-based pagination and make a collapsable tree for days in channels in networks (in users).

@twodayslate
Copy link

This is awesome! Thanks for working on this!

@DarthGandalf
Copy link
Member

Nothing happened with this in 2 years...

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.

5 participants