fix: version upgrade notification bug [CM-411]

[johnkim-det]

Ticket

CM-411

Description

Fix bug where version upgrade notification would not having expected styling applied. This is addressed by adding themeClass to the notification.warning call.

PR also includes:

• Fix for intermittent UIProvider error by moving code into a component that renders with the other global app components.
• Unrelated minor fix for console warning that I noticed while working on this.

Test Plan

Checklist

• Changes have been manually QA'd
• New features have been approved by the corresponding PM
• User-facing API changes have the "User-facing API Change" label
• Release notes have been added as a separate file under docs/release-notes/
  See Release Note for details.
• Licenses have been included for new code which was copied and/or modified from any external code

[netlify]

✅ Deploy Preview for determined-ui ready!

Name	Link
🔨 Latest commit	2f77e13
🔍 Latest deploy log	https://app.netlify.com/sites/determined-ui/deploys/6717ceaa03ebc900082c5ef0
😎 Deploy Preview	https://deploy-preview-10069--determined-ui.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[codecov]

Codecov Report

Attention: Patch coverage is 71.11111% with 26 lines in your changes missing coverage. Please review.

Project coverage is 50.65%. Comparing base (2356f91) to head (2f77e13).
Report is 38 commits behind head on main.

Files with missing lines	Patch %	Lines
webui/react/src/App.tsx	0.00%	12 Missing ⚠️
webui/react/src/pages/SignIn.tsx	0.00%	9 Missing ⚠️
webui/react/src/utils/browser.ts	16.66%	5 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #10069      +/-   ##
==========================================
- Coverage   54.62%   50.65%   -3.97%     
==========================================
  Files        1260      950     -310     
  Lines      157621   128964   -28657     
  Branches     3632     3635       +3     
==========================================
- Hits        86102    65328   -20774     
+ Misses      71385    63503    -7882     
+ Partials      134      133       -1     

Flag	Coverage Δ
harness	?
web	54.47% <71.11%> (+0.09%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
webui/react/src/components/JupyterLabButton.tsx	100.00% <100.00%> (ø)
webui/react/src/components/VersionChecker.tsx	100.00% <100.00%> (ø)
webui/react/src/utils/browser.ts	46.55% <16.66%> (-1.24%)	⬇️
webui/react/src/pages/SignIn.tsx	0.00% <0.00%> (ø)
webui/react/src/App.tsx	0.00% <0.00%> (ø)

... and 310 files with indirect coverage changes

[keita-determined]

found a bug in light mode

also could you add test plan?

[keita-determined]

i see many console errors

[keita-determined]

added few comments otherwise lgtm.
please double check (test) the behavior before merging it