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

Print cloud errors in console if present. #334

Merged
merged 2 commits into from
Nov 28, 2024

Conversation

Mati365
Copy link
Member

@Mati365 Mati365 commented Nov 28, 2024

Suggested merge commit message (convention)

Feat: Log cloud fetch errors to console.


Additional information

At this moment - nothing is being printed in console when the exception is thrown, so debugging the errors related to the CDN fetching might be time-consuming. I added logger.

Example log:

obraz

@coveralls
Copy link

Pull Request Test Coverage Report for Build 549be9e1-12e3-45ac-9fcb-0537265e9d84

Details

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 6d0d2bbb-0eda-4feb-bae5-2bd430a6ca45: 0.0%
Covered Lines: 84
Relevant Lines: 84

💛 - Coveralls

@Mati365 Mati365 merged commit 8ce672b into master Nov 28, 2024
5 checks passed
@Mati365 Mati365 deleted the ck/print-cdn-errors-in-console branch November 28, 2024 07:43
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.

3 participants