Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Bug fix: My Grants showing all grants #3513

Merged
merged 3 commits into from
Sep 17, 2024
Merged

Bug fix: My Grants showing all grants #3513

merged 3 commits into from
Sep 17, 2024

Conversation

greg-adams
Copy link
Contributor

@greg-adams greg-adams commented Sep 16, 2024 •
edited
Loading

Ticket #3512

Description

  • We were able to replicate this issue as a race condition from multiple grant tabs sharing a list of grants.
  • Resolutions for bugs on My Grants tab:
    1. Race condition for tabs sharing same grants fetching API endpoint.
    2. Duplicate route(s) causing match for first tab of MyGrantsView page and redirecting to page component without proper data (tabNames)
    3. Direct mutation of vuex state outside of commit/mutation calls

Screenshots / Demo Video

Testing

  • Steps to replicate: This issue can be reproduced by switching quickly between tabs on My Grants section where one return set is much larger than another (and returns from the server w/ delay).

Automated and Unit Tests

  • Added Unit tests

Manual tests for Reviewer

  • Added steps to test feature/functionality manually

Checklist

  • Provided ticket and description
  • Provided screenshots/demo
  • Provided testing information
  • Provided adequate test coverage for all new code
  • Added PR reviewers

@github-actions github-actions bot added bug Something isn't working javascript Pull requests that update Javascript code labels Sep 16, 2024
@github-actions GitHub Actions
Copy link

github-actions bot commented Sep 16, 2024 •
edited
Loading

QA Summary

QA Check Result
🌐 Client Tests ✅
🔗 Server Tests ✅
🤝 E2E Tests ✅
📏 ESLint ✅
🧹 TFLint ✅

Test Coverage

Coverage report for `packages/client`
St File % Stmts % Branch % Funcs % Lines Uncovered Line #s
🔴 All files 33.14 32.33 30.08 34.72
🔴  src 0 100 100 0
🔴   App.vue 0 100 100 0 2-9
🔴  src/arpa_reporter 0 100 100 0
🔴   App.vue 0 100 100 0 2-13
🟡  ...ter/components 58.58 48 46.8 59.78
🟡   AlertBox.vue 80 75 50 80 13
🔴   ...oadButton.vue 20 0 0 20 2-7,38-67
🟢   ...ileButton.vue 100 100 100 100
🟢   ...ttonSmall.vue 100 100 100 100
🟢   ...mplateBtn.vue 100 100 100 100
🟡   ...avigation.vue 67.74 63.63 52.63 67.74 ...13-219,228-235
🔴   StandardForm.vue 45 50 41.66 45.45 ...24-128,135-157
🟢  ...porter/helpers 84.61 79.48 87.5 84.61
🟢   form-helpers.js 84.21 79.48 85.71 84.21 7,16,25,81-83
🟢   short-uuid.js 100 100 100 100
🔴  ...eporter/router 0 0 0 0
🔴   index.js 0 0 0 0 20-131
🔴  ...reporter/store 3.92 0 2.17 4.12
🔴   index.js 3.92 0 2.17 4.12 11-14,32-261
🔴  ...reporter/views 40.98 25.13 41.37 42.91
🟢   AgenciesView.vue 100 0 100 100 16
🔴   AgencyView.vue 40.74 36.36 50 45.83 53-62,70-96
🔴   HomeView.vue 26.66 34.14 36.36 26.66 36-66,113,137-207
🔴   LoginView.vue 29.62 36.36 20 30.76 1,4,19-34,72-100
🔴   ...plateView.vue 27.02 35.71 45.45 25 ...2,30-37,69-113
🔴   ...ploadView.vue 24.24 16.66 33.33 26.66 1,30-31,116-144
🔴   ...eriodView.vue 48 18.18 75 52.17 64-90
🟡   ...riodsView.vue 57.57 28.57 60 59.37 101,132,149-171
🔴   ...pientView.vue 40 10.52 30.76 41.02 73-93,110-152
🔴   ...ientsView.vue 48.48 8.33 46.66 53.33 ...46,165-190,203
🟡   UploadView.vue 61.53 43.75 62.5 61.36 ...41-442,448-449
🔴   UploadsView.vue 16.66 0 0 18.18 59,110-287
🔴   UserView.vue 46.34 33.33 68.75 48.64 84,97-137
🟡   UsersView.vue 60 7.69 62.5 66.66 91-92,105-107,123
🔴   ...ationView.vue 37.03 18.18 26.66 41.66 ...19,238,246-270
🔴  src/components 45.55 37.34 47.27 46.53
🔴   ...vityTable.vue 18.75 0 28.57 19.35 115-185
🟡   BaseLayout.vue 69.56 53.84 60 69.56 172,219-231
🔴   CopyButton.vue 14.28 0 0 14.28 5,37-49
🟢   ...tActivity.vue 81.39 62.5 92.3 81.39 ...00,113-118,149
🔴   GrantsTable.vue 45.03 38.77 45 46.85 ...51-455,463-543
🔴   ...dUploader.vue 24.24 33.33 37.5 24.24 16,25,74-111
🔴   SearchFilter.vue 40.74 28 37.5 40 ...52,64,67,72-82
🔴   ShareGrant.vue 44.11 71.42 23.07 45.45 ...03-107,126-147
🔴   UserAvatar.vue 40 25 100 37.5 31-37
🔴  ...ponents/Modals 26.64 43.08 31.91 30.18
🔴   ...anization.vue 17.39 65.62 30 22.22 1-14,149-178
🔴   AddTeam.vue 45.45 55.55 61.53 57.69 ...04,210,222-245
🔴   AddUser.vue 36.66 66.66 60 37.5 ...40,145,148-176
🔴   ...anization.vue 15.38 64.28 14.28 16.66 1-15,58-78
🔴   EditTeam.vue 18.18 26.43 33.33 20.83 ...29,208,216-301
🔴   EditUser.vue 21.05 66.66 25 22.22 1,101-128
🔴   ...ilsLegacy.vue 22.03 0 0 24.07 131,177,205-369
🟡   ImportTeams.vue 50 41.17 50 53.33 28,64-69,81-82
🔴   ImportUsers.vue 42.85 60 40 46.15 29,65-80
🔴   ...archPanel.vue 27.9 15.78 23.52 28.57 ...68-178,211-255
🔴   SearchPanel.vue 21.62 63.26 32 27.58 ...77-380,386-458
🔴  src/helpers 15.27 17.92 14.63 15.94
🟢   constants.js 100 100 100 100
🟢   currency.js 100 100 100 100
🟡   dates.js 66.66 100 33.33 100
🔴   fetchApi.js 6 13.79 5.26 6.12 10-12,20-132
🔴   filters.js 4 0 0 4.54 19-51
🔴   form-helpers.js 0 0 0 0 5-82
🟡   gtag.js 77.77 90 75 77.77 12,51
🔴   ...patWarning.js 0 0 0 0 39-61
🟢  ...s/featureFlags 83.33 100 66.66 83.33
🟡   index.js 50 100 50 50 8,16
🟢   utils.js 100 100 100 100
🔴  src/mixin 20 0 28.57 20
🔴   ...zableTable.js 20 0 28.57 20 16-31,36-37,42
🔴  src/router 18.91 14.28 11.11 18.91
🔴   index.js 18.91 14.28 11.11 18.91 ...76-177,181-200
🟢  src/store 100 100 100 100
🟢   index.js 100 100 100 100
🔴  src/store/modules 3.07 0 4.57 3.21
🔴   agencies.js 5.26 100 8.33 5.55 13-70
🔴   alerts.js 20 100 20 20 10-24
🔴   grants.js 1.12 0 1 1.16 61-419
🔴   organization.js 33.33 100 33.33 33.33 21-25
🔴   roles.js 20 100 20 25 13-22
🔴   tenants.js 11.11 100 14.28 12.5 13-32
🔴   users.js 2.43 0 4.76 2.5 17-100
🔴  src/views 42.95 35.16 32.45 44.67
🔴   ...orterView.vue 25.58 51.85 18.18 26.82 ...,62,84,109-151
🟡   ...boardView.vue 50 17.64 50 52 89-98,114-125
🔴   ...tailsView.vue 33.68 13.33 17.07 34.4 ...89-428,434-455
🟢   GrantsView.vue 100 100 100 100
🔴   LoginView.vue 34.61 33.33 22.22 36 1,22,57,109-136
🟡   MyGrantsView.vue 77.77 66.66 66.66 77.77 1,69
🟡   ...ofileView.vue 77.27 75 42.85 77.27 1,32,63,130-134
🟢   NotFoundView.vue 100 100 100 100
🔴   ...tionsView.vue 47.05 57.14 41.66 53.33 ...97-100,114-118
🔴   ...ivityView.vue 46.42 23.8 43.75 46.42 ...01,114,120-134
🔴   TeamsView.vue 44.44 88.88 41.66 53.33 1,58,142,156-163
🟡   UsersView.vue 50 66.66 36.36 53.84 ...16-121,133-139
Coverage report for `packages/server`
St File % Stmts % Branch % Funcs % Lines Uncovered Line #s
🟡 All files 59.38 52.22 54.85 59.47
🟢  src 81.63 33.33 60 81.63
🟢   configure.js 81.63 33.33 60 81.63 42,61-68,97-99
🟢  src/arpa_reporter 98.75 66.66 100 98.75
🟢   configure.js 97.36 40 100 97.36 36
🟢   environment.js 100 100 100 100
🟢   use-request.js 100 100 100 100
🔴  src/arpa_reporter/db 38.58 32.92 44.44 40.16
🔴   arpa-subrecipients.js 13.15 4.34 15.38 14.28 23-92
🔴   reporting-periods.js 37.2 46.87 40 38.09 46,77-156
🟢   settings.js 100 83.33 100 100 13
🟡   uploads.js 50 28.57 52.38 51.42 18-29,84,99-124,141-150
🔴  src/arpa_reporter/lib 29.57 33.08 34.56 28.46
🟢   arpa-ec-codes.js 100 100 100 100
🔴   audit-report.js 21.44 19.35 24.19 21.32 ...28-529,554-684,732-758
🟡   ensure-async-context.js 75 100 50 100
🟢   format.js 90.62 90 90 91.3 41-42
🟡   log.js 75 50 50 75 13,25
🟡   preconditions.js 66.66 33.33 100 66.66 3
🔴   spreadsheet.js 9.09 0 0 9.09 15-32
🟢   validation-error.js 85.71 100 50 85.71 16
🔴  src/arpa_reporter/routes 40 14.92 14.28 40.6
🔴   agencies.js 22.58 0 0 23.33 13-21,26-53
🟡   application_settings.js 75 100 0 75 10-11
🟡   audit-report.js 68.91 58.33 100 68.91 57-58,64-78,100-116
🟢   exports.js 81.42 83.33 100 81.42 61-75,98-99
🔴   reporting-periods.js 20 0 0 20.43 ...25-137,143-149,154-180
🔴   subrecipients.js 23.8 0 0 23.8 12-13,17-27,31-48,52-63
🔴   uploads.js 28.28 7.89 9.09 29.16 ...33-154,164-166,173-180
🔴   users.js 19.6 0 0 20 15-35,39-44,48-81
🔴  src/arpa_reporter/services 44.32 34.18 46.42 44.6
🔴   generate-arpa-report.js 36.86 2.77 50 37.24 ...-975,984-997,1071-1138
🔴   get-template.js 21.62 0 0 21.62 18-79
🟡   persist-upload.js 68.6 90 69.56 68.67 ...58-200,221-235,273-295
🔴   records.js 20.75 0 11.11 21.15 38-204,221-276
🔴   revalidate-uploads.js 37.5 100 0 37.5 5-14
🔴   validate-upload.js 43.56 57.21 37.03 44.2 ...48,367,389,407-684,699
🟢   validation-rules.js 98.18 90 90.9 100 157,173
🟡  src/db 74.26 71.42 68.42 74.29
🟢   connection.js 100 50 100 100 6
🟢   constants.js 100 100 100 100
🟡   helpers.js 75 83.33 50 75 5,21-22
🟢   index.js 82.4 78.54 82.08 82.35 ...48-1414,1596-1597,1604
🟢   saved_search_migration.js 92 88.23 71.42 93.61 5,69,134
🔴   tenant_creation.js 10.58 2.7 0 11.11 15-40,48-210,220
🔴  src/db/arpa_reporter_db_shims 23.68 0 0 23.68
🔴   agencies.js 22.22 100 0 22.22 11-51
🔴   users.js 25 0 0 25 12-62
🟢  src/lib 86.24 79.44 91.42 86.2
🟢   access-helpers.js 93.54 89.18 100 93.54 96-97,102-103
🟢   agencyImporter.js 90.38 88.46...[Comment body truncated]

@github-actions GitHub Actions
Copy link

github-actions bot commented Sep 16, 2024 •
edited
Loading

Terraform Summary

Step Result
🖌 Terraform Format & Style ✅
⚙️ Terraform Initialization ✅
🤖 Terraform Validation ✅
📖 Terraform Plan ✅

Hint: If "Terraform Format & Style" failed, run terraform fmt -recursive from the terraform/ directory and commit the results.

Output

Validation Output 
stdout:
Success! The configuration is valid.


-------------------------------------
stderr:
Plan Summary
CHANGE RESOURCE
add module.website.aws_s3_object.origin_dist_artifact["assets/ActivityTable-DfKIQ_wW.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/ActivityTable-DfKIQ_wW.js.map"]
module.website.aws_s3_object.origin_dist_artifact["assets/ArpaAnnualPerformanceReporterView-DgxIsI5n.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/ArpaAnnualPerformanceReporterView-DgxIsI5n.js.map"]
module.website.aws_s3_object.origin_dist_artifact["assets/DashboardView-COUUTYvC.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/DashboardView-COUUTYvC.js.map"]
module.website.aws_s3_object.origin_dist_artifact["assets/GrantDetailsLegacy-oWUj10By.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/GrantDetailsLegacy-oWUj10By.js.map"]
module.website.aws_s3_object.origin_dist_artifact["assets/GrantDetailsView-DE7cIs38.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/GrantDetailsView-DE7cIs38.js.map"]
module.website.aws_s3_object.origin_dist_artifact["assets/GrantsTable-B62MfCYJ.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/GrantsTable-B62MfCYJ.js.map"]
module.website.aws_s3_object.origin_dist_artifact["assets/GrantsView-BByRHd4f.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/GrantsView-BByRHd4f.js.map"]
module.website.aws_s3_object.origin_dist_artifact["assets/MyGrantsView-DcMMsUyl.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/MyGrantsView-DcMMsUyl.js.map"]
module.website.aws_s3_object.origin_dist_artifact["assets/MyProfileView-0hyvjOFZ.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/MyProfileView-0hyvjOFZ.js.map"]
module.website.aws_s3_object.origin_dist_artifact["assets/NotFoundView-BHa6zy1b.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/NotFoundView-BHa6zy1b.js.map"]
module.website.aws_s3_object.origin_dist_artifact["assets/OrganizationsView-MJuFHxhn.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/OrganizationsView-MJuFHxhn.js.map"]
module.website.aws_s3_object.origin_dist_artifact["assets/RecentActivityView-BIMaKXlV.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/RecentActivityView-BIMaKXlV.js.map"]
module.website.aws_s3_object.origin_dist_artifact["assets/RecordUploader-Cu0fIHyN.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/RecordUploader-Cu0fIHyN.js.map"]
module.website.aws_s3_object.origin_dist_artifact["assets/TeamsView-DidaSaMN.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/TeamsView-DidaSaMN.js.map"]
module.website.aws_s3_object.origin_dist_artifact["assets/UsersView-CbNN3Gbl.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/UsersView-CbNN3Gbl.js.map"]
module.website.aws_s3_object.origin_dist_artifact["assets/arpa_reporter-Cyzx1SnQ.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/arpa_reporter-Cyzx1SnQ.js.map"]
module.website.aws_s3_object.origin_dist_artifact["assets/form-helpers-Bjl_m51z.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/form-helpers-Bjl_m51z.js.map"]
module.website.aws_s3_object.origin_dist_artifact["assets/main-BDRfdkA7.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/main-BDRfdkA7.js.map"]
module.website.aws_s3_object.origin_dist_artifact["assets/style-DF53RXBe.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/style-DF53RXBe.js.map"]
update module.api.aws_ecs_service.default[0]
module.api.module.grant_digest_scheduled_task.aws_iam_role_policy.default[0]
module.api.module.grant_digest_scheduled_task.aws_scheduler_schedule.default[0]
module.arpa_audit_report.aws_ecs_service.default
module.arpa_treasury_report.aws_ecs_service.default
module.consume_grants.aws_ecs_service.default
module.website.aws_s3_object.deploy-config[0]
module.website.aws_s3_object.origin_dist_artifact["arpa_reporter/index.html"]
module.website.aws_s3_object.origin_dist_artifact["index.html"]
recreate module.api.aws_ecs_task_definition.default[0]
module.arpa_audit_report.aws_ecs_task_definition.consumer
module.arpa_treasury_report.aws_ecs_task_definition.consumer
module.consume_grants.aws_ecs_task_definition.consume_grants
delete module.website.aws_s3_object.origin_dist_artifact["assets/ActivityTable-gtIuOQTO.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/ActivityTable-gtIuOQTO.js.map"]
module.website.aws_s3_object.origin_dist_artifact["assets/ArpaAnnualPerformanceReporterView-C4PjYNdI.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/ArpaAnnualPerformanceReporterView-C4PjYNdI.js.map"]
module.website.aws_s3_object.origin_dist_artifact["assets/DashboardView-CYQNq0GR.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/DashboardView-CYQNq0GR.js.map"]
module.website.aws_s3_object.origin_dist_artifact["assets/GrantDetailsLegacy-CpLGXnLU.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/GrantDetailsLegacy-CpLGXnLU.js.map"]
module.website.aws_s3_object.origin_dist_artifact["assets/GrantDetailsView-DvNnLzhH.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/GrantDetailsView-DvNnLzhH.js.map"]
module.website.aws_s3_object.origin_dist_artifact["assets/GrantsTable-DWAsBNBl.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/GrantsTable-DWAsBNBl.js.map"]
module.website.aws_s3_object.origin_dist_artifact["assets/GrantsView-C6fNOnLr.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/GrantsView-C6fNOnLr.js.map"]
module.website.aws_s3_object.origin_dist_artifact["assets/MyGrantsView-BwmgVC6M.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/MyGrantsView-BwmgVC6M.js.map"]
module.website.aws_s3_object.origin_dist_artifact["assets/MyProfileView-CqpU4SfF.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/MyProfileView-CqpU4SfF.js.map"]
module.website.aws_s3_object.origin_dist_artifact["assets/NotFoundView-B5d8atcT.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/NotFoundView-B5d8atcT.js.map"]
module.website.aws_s3_object.origin_dist_artifact["assets/OrganizationsView-1jpYFwk-.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/OrganizationsView-1jpYFwk-.js.map"]
module.website.aws_s3_object.origin_dist_artifact["assets/RecentActivityView-DTrDUpfd.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/RecentActivityView-DTrDUpfd.js.map"]
module.website.aws_s3_object.origin_dist_artifact["assets/RecordUploader-C9kHRIk7.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/RecordUploader-C9kHRIk7.js.map"]
module.website.aws_s3_object.origin_dist_artifact["assets/TeamsView-CBavhsAj.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/TeamsView-CBavhsAj.js.map"]
module.website.aws_s3_object.origin_dist_artifact["assets/UsersView-BTRmD9Nn.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/UsersView-BTRmD9Nn.js.map"]
module.website.aws_s3_object.origin_dist_artifact["assets/arpa_reporter-C2dMCqN9.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/arpa_reporter-C2dMCqN9.js.map"]
module.website.aws_s3_object.origin_dist_artifact["assets/form-helpers-BkFqWbxB.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/form-helpers-BkFqWbxB.js.map"]
module.website.aws_s3_object.origin_dist_artifact["assets/main-DmO4gY9F.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/main-DmO4gY9F.js.map"]
module.website.aws_s3_object.origin_dist_artifact["assets/style-CBuUiTyI.js"]
module.website.aws_s3_object.origin_dist_artifact["assets/style-CBuUiTyI.js.map"]

Pusher: @greg-adams, Action: pull_request_target, Workflow: Continuous Integration

@greg-adams greg-adams closed this Sep 17, 2024
@greg-adams greg-adams reopened this Sep 17, 2024
@greg-adams greg-adams self-assigned this Sep 17, 2024
@greg-adams greg-adams changed the title Bug fix: resolve visible console errors Bug fix: My Grants showing all grants Sep 17, 2024
as1729
as1729 reviewed Sep 17, 2024
View reviewed changes
packages/client/src/store/modules/grants.js
currentPage, perPage, orderBy, orderDesc,
}) {
const pagination = { currentPage, perPage };
const ordering = { orderBy, orderDesc };
const filters = { ...this.state.grants.searchFormFilters };
const { criteriaQuery, paginationQuery, orderingQuery } = buildGrantsNextQuery({ filters, ordering, pagination });

// Avoid race conditions for tabs sharing grant fetching
const requestId = state.grantsRequestId + 1;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

suggestion: update this to a uuid. my concern is there is a possibility of collision if the state information here is stale and the IDs are too close to each other. I feel like using a uuid might eliminate the possibility.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 will update in next pr

as1729
as1729 approved these changes Sep 17, 2024
View reviewed changes
Copy link
Contributor

@as1729 as1729 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

my suggestion here is non-blocking as the solution works well

@greg-adams greg-adams merged commit 835d833 into main Sep 17, 2024
22 checks passed
@greg-adams greg-adams deleted the bug/my-grants-tab branch September 17, 2024 20:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@as1729 as1729 as1729 approved these changes

@TylerHendrickson TylerHendrickson Awaiting requested review from TylerHendrickson

Assignees

@greg-adams greg-adams

Labels
bug Something isn't working javascript Pull requests that update Javascript code
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@greg-adams @as1729