Skip to content

[v6] utils Logging #2526

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

Closed
wants to merge 15 commits into from
Closed

[v6] utils Logging #2526

wants to merge 15 commits into from

Conversation

xzilja
Copy link
Contributor

@xzilja xzilja commented Aug 28, 2019

Work in progress:

  • Adjust colours
  • Add logging to native events
  • Disable logging by default (from test app.js)
  • Better params formmating in info logger

I tried to make it nice ¯_(ツ)_/¯

Screenshot 2019-09-04 at 17 13 13

@codecov
Copy link

codecov bot commented Aug 28, 2019

Codecov Report

Merging #2526 into master will increase coverage by 0.61%.
The diff coverage is 72.73%.

@@            Coverage Diff             @@
##           master    #2526      +/-   ##
==========================================
+ Coverage   88.78%   89.39%   +0.61%     
==========================================
  Files         111      110       -1     
  Lines        3431     3279     -152     
==========================================
- Hits         3046     2931     -115     
+ Misses        385      348      -37

@Salakar Salakar changed the title [WIP] utils Logging [v6] [WIP] utils Logging Aug 29, 2019
@xzilja xzilja changed the title [v6] [WIP] utils Logging [v6] utils Logging Sep 4, 2019
enableEventLogging: true,
});

// config = { enableMethodLogging: true, enableEventLogging: ture }
Copy link
Member

Choose a reason for hiding this comment

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

ture -> true


### Enable logging

You can enable logging for native method calls and event streams
Copy link
Member

Choose a reason for hiding this comment

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

Period/semi-colon on the end


### Log out custom information

You can also use our built in logger to log your custom information
Copy link
Member

Choose a reason for hiding this comment

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

Period/semi-colon on the end

@@ -240,6 +240,7 @@ export namespace ReactNativeFirebase {
*/
export namespace Utils {
import FirebaseModule = ReactNativeFirebase.FirebaseModule;
type LoggerConfig = { enableMethodLogging: boolean; enableEventLogging: boolean };
Copy link
Member

Choose a reason for hiding this comment

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

This will get added to the docs so worth having a quick description above it.

@Salakar Salakar added this to the v6.1.0 milestone Oct 7, 2019
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.

4 participants