Skip to content
This repository was archived by the owner on May 20, 2026. It is now read-only.

[Issue-51] Log warnings when fallback image is delivered.#52

Merged
mpospese merged 1 commit into
mainfrom
issue/51-log-warnings-on-image-fallback
Mar 17, 2023
Merged

[Issue-51] Log warnings when fallback image is delivered.#52
mpospese merged 1 commit into
mainfrom
issue/51-log-warnings-on-image-fallback

Conversation

@mpospese

Copy link
Copy Markdown
Contributor

Introduction

When either ImageAsset or SystemImage fails to load an image and delivers the fallback image instead we should log a warning to the console to alert the developer.

Purpose

Fix #51: Optionally log warning to console when delivering the fallback image.

Scope

  • Add logging to ImageAsset.image
  • Add logging to SystemImage.image
  • Add a logger singleton plus a flag to optionally silence it.
  • Unit tests

📈 Coverage

Code

100%
image

Documentation

100%
image

@mpospese mpospese added the enhancement New feature or request label Mar 16, 2023
@mpospese mpospese self-assigned this Mar 16, 2023
@mpospese
mpospese merged commit 06398c4 into main Mar 17, 2023
@mpospese
mpospese deleted the issue/51-log-warnings-on-image-fallback branch March 17, 2023 05:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request

Development

Successfully merging this pull request may close these issues.

Log warnings in ImageAsset.image and SystemImage.image when loadImage() fails

2 participants