Closed ksmontville closed 3 weeks ago
Status | Category | Percentage | Covered / Total |
---|---|---|---|
🔵 | Lines | 0.96% | 73 / 7564 |
🔵 | Statements | 0.89% | 74 / 8311 |
🔵 | Functions | 0.75% | 14 / 1851 |
🔵 | Branches | 0.44% | 20 / 4538 |
Visit the preview URL for this PR (updated for commit 2bc49d2):
https://roar-staging--pr821-fix-app-public-names-xhh9dm2q.web.app
(expires Mon, 30 Sep 2024 18:44:31 GMT)
🔥 via Firebase Hosting GitHub Action 🌎
Sign: 2631e9c58fd0104ecbfddd72a62245ddac467460
Passed #7087
•
2bc49d2bfb: E2E Tests for PR 821 "CU-8689ubqge fix/App Public Names in Reports" from commit ...
Project |
roar-dashboard-e2e
|
Branch Review |
fix/app-public-names
|
Run status |
Passed #7087
|
Run duration | 03m 30s |
Commit |
2bc49d2bfb: E2E Tests for PR 821 "CU-8689ubqge fix/App Public Names in Reports" from commit ...
|
Committer | Kyle |
View all properties for this run ↗︎ |
Test results | |
---|---|
Failures |
0
|
Flaky |
0
|
Pending |
0
|
Skipped |
0
|
Passing |
13
|
View all changes introduced in this branch ↗︎ |
This pull request was deployed and Sentry observed the following issues:
<object>.actions.updateTasksDictionary(src/stor...
View Issue<object>.actions.updateTasksDictionary(src/stor...
View Issue<object>.actions.updateTasksDictionary(src/stor...
View Issue<object>.actions.updateTasksDictionary(src/stor...
View IssueinitNewFirekit(src/firebaseInit)
View IssueDid you find this useful? React with a 👍 or 👎
Proposed changes
Fix for
taskId
showing in place ofpublicName
in reports.Major Changes:
tasksDictionary
within authstore, rather than relying on firekit method. The logic mimics the logic found within the firekit method. This is because thetasksDictionary
object was often being accessed before the firekit method could populate it.authStore.updateTasksDictionary()
toonUpdated()
hook, helping to ensure thattasksDictionary
is defined when the DOM or auth state changes.HomeAdministrator.vue
,HomeParticipant.vue
, andConsentModal.vue
withinHomeSelector.vue
instead of dynamic imports. The dynamic imports were causing issues with component rendering when re-directing from Clever sign in. There is likely a better fix for this in the future.Types of changes
What types of changes does this pull request introduce?
Checklist
Justification of missing checklist items
Further comments