← Back to team overview

sts-sponsors team mailing list archive

[Merge] ~petermakowski/maas-site-manager:error-handling-MAASENG-1389 into maas-site-manager:main

 

The proposal to merge ~petermakowski/maas-site-manager:error-handling-MAASENG-1389 into maas-site-manager:main has been updated.

Description changed to:

QA Steps

- run the site locally
- open developer tools
- verify that the environment (development), version number and git hash are logged into the console

- add a the following lines to main.tsx:
setTimeout(() => {
 throw new Error("sentry error")
}, 3000);
- open developer tools and go to network tab
- refresh the page
- verify that an error request has made for URL starting with https://sentry.is.canonical.com/api/60/store/
- verify the request has been made with the correct data  by looking at the request payload - it should include relative url, e.g. "/sites" as the url tag parameter

run yarn upgrade-all
and ensure that @sentry/browser version remained unchanged

verify that maas site manager instance is setup and you can access it: https://sentry.is.canonical.com/canonical/maas-site-manager/

For more details, see:
https://code.launchpad.net/~petermakowski/maas-site-manager/+git/site-manager/+merge/441092
-- 
Your team MAAS Committers is requested to review the proposed merge of ~petermakowski/maas-site-manager:error-handling-MAASENG-1389 into maas-site-manager:main.



References