[$150] Connect project listing or project details page does not load when /notifications/list call timesout

[vikasrohit]

Expected behavior

Should load the app as normal, only thing that should not be working is the notifications panel

Actual behavior

It does not render any thing if there is a timeout in /notifications call

Steps to reproduce the problem

• TBD.
• TBD.
• TBD.

Screenshot/screencast

--

Environment

• OS:
• Browser (w/version):
• User role (client, copilot or manager):
• Account used:

[vikasrohit]

@maxceem could you please take a look when you have time? It is not that urgent but good to have fixed as soon as possible because we are seeing such errors now too often.

[maxceem]

@vikasrohit sure, would you like it to be fixed before form wizard/dependant question is done or after?

[vikasrohit]

If it does not take more than 30 minutes, I would like to get it first as we are seeing issues on production.

[maxceem]

@vikasrohit I cannot reproduce the issue.

I simulate the timeout error by throwing error inside getNotifications in 30 seconds:

const getNotifications = () => {
  return new Promise(() => {
    setTimeout(() => {
      throw new Error('error loading getNotifications')
    }, 30000)
  })
  // return axios.get(`${TC_NOTIFICATION_URL}/list?read=false&limit=${NOTIFICATIONS_LIMIT}`)
  //  .then(resp => prepareNotifications(resp.data.items))
}

But both project listing and project details pages are loaded without any issues:

I guess both these pages could fail to be loaded if metadata requrests is failed due to timeout. It would help a lot if someone who next time experince such issue will share the screenshot of the browser console.

[maxceem]

@vikasrohit have seen this issue lately or maybe we can close it for now?

[vikasrohit]

Yes, I have seen it yesterday. Console just says timeout because list notifications call took more than 20 seconds.

I think to simulate you need to pass lesser timeout not greater. Try setting it to 100ms to simulate the timeout.