leancodepl / leancodepl/flutter_corelibrary

[login_client] Use `package:logging` instead of a logger callback

Open
#464 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

p: login_client
Dominant language
Dart
Stars
89
Forks
12
Avg merge
2d 7h
Merged PRs (30d)
2

Description

Consider migrating from a simple void Function(String) for logging, and use the full Logger API.

  • Currently, users of this package have no way of filtering logs (e.g., by severity)
  • Using Logger would provide a sane default configuration, instead of print()
  • The logs would contain exceptions and stack traces

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by locating the login_client code that accepts the logger callback and review how its current print-based logging is configured. Check the package:logging API and dependency setup; done means logging uses Logger with severity filtering and preserves exceptions and stack traces.

Written by the indexing model from the issue text.

Assessment

Tech stack
dart, flutter
Domain
authentication
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.