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

[Bug]: Notifications - Error fetching from Contentful after 3 retries: Error: Fetch failed with status: 404 #25748

Closed
seaona opened this issue Jul 10, 2024 · 6 comments
Labels
area-performance Issues relating to slowness of app, cpu usage, and/or blank screens. regression-RC-12.0.0 Sev2-normal Normal severity; minor loss of service or inconvenience. team-notifications Notifications team type-bug

Comments

@seaona
Copy link
Contributor

seaona commented Jul 10, 2024

Describe the bug

Whenever I enable notifications I see the following background error Error fetching from Contentful after 3 retries: Error: Fetch failed with status: 404and it takes several seconds for the loading spinner to disappear (not sure if it's related).

Expected behavior

No error and good performance

Screenshots/Recordings

Screenshot from 2024-07-10 18-04-55

notifications-perf-and-error.mp4

Steps to reproduce

  1. Onboard
  2. Click Enable notifications
  3. See error in the background/service worker console

Error messages or log output

No response

Version

12.0.0

Build type

None

Browser

Chrome

Operating system

Linux

Hardware wallet

No response

Additional context

No response

Severity

No response

@seaona seaona added type-bug team-notifications Notifications team regression-RC-12.0.0 area-performance Issues relating to slowness of app, cpu usage, and/or blank screens. labels Jul 10, 2024
@sleepytanya
Copy link
Contributor

Different error on 12.1.0, appears every time I turn notifications ON and OFF:

Screen.Recording.2024-07-10.at.18.26.48.mov
Screenshot 2024-07-10 at 18 28 08

@Prithpal-Sooriya
Copy link
Contributor

TY. Jira Ticket created here. Our team will investigate.

@Prithpal-Sooriya
Copy link
Contributor

It may be that this API call to contentful is missing some environment variables on dev, but has been added to our CI pipeline/releases.

We will find a better way to handle this.

@Prithpal-Sooriya
Copy link
Contributor

A PR has been made in core libraries.

We are in the process of getting the extension to use the core libraries, but can backport this into extension.

@Prithpal-Sooriya
Copy link
Contributor

Prithpal-Sooriya commented Jul 22, 2024

This was backported here: #25979

@Prithpal-Sooriya
Copy link
Contributor

This is fixed in v12.0.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-performance Issues relating to slowness of app, cpu usage, and/or blank screens. regression-RC-12.0.0 Sev2-normal Normal severity; minor loss of service or inconvenience. team-notifications Notifications team type-bug
Projects
Archived in project
Development

No branches or pull requests

4 participants