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

Added log module #20

Closed
wants to merge 5 commits into from
Closed

Added log module #20

wants to merge 5 commits into from

Conversation

AntonHolovin
Copy link
Contributor

Hi,

I've ported log module from u2020 app. It has dependency on Jake Wharton's Timber (https://github.com/JakeWharton/timber). If user wants to use it he should add some lines to Application class. I also updated example app.

User can see logs on the devise. He also can share log file. So he has same functions from u2020 app.

screenshot_2015-08-09-19-34-29

@palaima
Copy link
Owner

palaima commented Aug 13, 2015

Is here a problem with PR, thats why you have closed?
Currently i do not have enough time, but on weekend I could review and merge your PR if you want.
And Thanks for contributing. Its really awesome module!!

@AntonHolovin
Copy link
Contributor Author

I made an error because that pull request is designed for master branch. I can reopen it or create new request to dev branch. So I decided to edit description and create pull request to dev branch.

@palaima
Copy link
Owner

palaima commented Aug 13, 2015

👍

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.

None yet

2 participants