Skip to content
This repository has been archived by the owner on May 14, 2024. It is now read-only.

Update error handlings #1021

Merged
merged 8 commits into from
Apr 5, 2023
Merged

Update error handlings #1021

merged 8 commits into from
Apr 5, 2023

Conversation

abhinavkgrd
Copy link
Member

@abhinavkgrd abhinavkgrd commented Apr 5, 2023

Description

  • fix syncWithRemote logging handled errors
  • add txt and HTML to known non-media file formats to ignore logging their upload fail error
  • ignore msrc already set warning in case new url is same as old and ignore it as it is possible that component render can call the function multiple time, but its harmless and can be ignored.
  • added checks to prevent ml DB initialization on the browser
  • handle auth key not set on server and remove logging for the case

Test Plan

tested locally

@cloudflare-pages
Copy link

cloudflare-pages bot commented Apr 5, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: ea9e2f0
Status: ✅  Deploy successful!
Preview URL: https://822e4433.bada-frame.pages.dev
Branch Preview URL: https://update-error-handlings.bada-frame.pages.dev

View logs

@abhinavkgrd abhinavkgrd marked this pull request as ready for review April 5, 2023 13:04
@abhinavkgrd abhinavkgrd requested a review from ua741 April 5, 2023 13:04
@abhinavkgrd abhinavkgrd merged commit 85e75ac into main Apr 5, 2023
@abhinavkgrd abhinavkgrd deleted the update-error-handlings branch April 5, 2023 13:29
@abhinavkgrd abhinavkgrd mentioned this pull request Apr 10, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants