Eliminate use of v-html when rendering node names and descriptions

ptgolden commented 5 days ago

Related issues

Closes #902

Summary

Adds a new component, <AppNodeText> which safely renders text containing <i>, <sup>, and <a> tags, while also leaving arbitrary text \<enclosed in brackets> untouched.
Removes the usage of the unsafe v-html directive when rendering nodes, replacing it with <AppNodeText>

Checks

[x] All tests have passed (or issues created for failing tests)

netlify[bot] commented 5 days ago

Deploy Preview for monarch-app ready!

Name	Link
Latest commit	161a149292335a9fa2fcc15b4ecf9e55a6368f88
Latest deploy log	https://app.netlify.com/sites/monarch-app/deploys/673f4246277cfb0008a5e227
Deploy Preview	https://deploy-preview-908--monarch-app.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[bot] commented 5 days ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 71.14%. Comparing base (ce0851b) to head (161a149). Report is 5 commits behind head on main.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #908 +/- ## ========================================== - Coverage 71.37% 71.14% -0.24% ========================================== Files 91 91 Lines 3148 3136 -12 ========================================== - Hits 2247 2231 -16 - Misses 901 905 +4 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests
JS Bundle Analysis - Avoid shipping oversized bundles

monarch-initiative / monarch-app