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.

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

893 upgrade client dependencies reopened #1028

Conversation

atomicgamedeveloper
Copy link
Contributor

Pull Request Template

Title

New PR for #945.

Type of Change

  • New feature
  • Bug fix
  • Documentation update
  • Refactoring
  • Security patch
  • UI/UX improvement

Description

Please refer to old PR.

Testing

Please refer to old PR.

Impact

Please refer to old PR.

Additional Information

Please refer to old PR.

Checklist

  • My code adheres to the coding and style guidelines of the project.
  • I have added tests for all the new code and any changes made to
    existing code.
  • I have made corresponding changes to the documentation.

Copy link

codeclimate bot commented Nov 1, 2024

Code Climate has analyzed commit 680c487 and detected 0 issues on this pull request.

View more on Code Climate.

@prasadtalasila prasadtalasila marked this pull request as ready for review November 1, 2024 16:18
Copy link

codecov bot commented Nov 1, 2024

Codecov Report

Attention: Patch coverage is 38.23529% with 21 lines in your changes missing coverage. Please review.

Project coverage is 73.52%. Comparing base (7a07bc2) to head (05f53e7).
Report is 1 commits behind head on feature/distributed-demo.

Files with missing lines Patch % Lines
client/src/util/auth/Authentication.ts 20.00% 17 Missing and 3 partials ⚠️
client/src/route/library/Library.tsx 50.00% 1 Missing ⚠️
Additional details and impacted files
@@                      Coverage Diff                      @@
##           feature/distributed-demo    #1028       +/-   ##
=============================================================
+ Coverage                      0.00%   73.52%   +73.52%     
=============================================================
  Files                            11       59       +48     
  Lines                            99     1099     +1000     
  Branches                          4       78       +74     
=============================================================
+ Hits                              0      808      +808     
- Misses                           95      214      +119     
- Partials                          4       77       +73     
Files with missing lines Coverage Δ
client/src/page/Menu.tsx 95.45% <100.00%> (ø)
client/src/page/MenuToolbar.tsx 93.33% <100.00%> (ø)
...preview/route/digitaltwins/DigitalTwinsPreview.tsx 83.33% <100.00%> (ø)
...view/route/digitaltwins/execute/pipelineHandler.ts 89.65% <ø> (ø)
client/src/preview/util/gitlabDigitalTwin.ts 38.29% <ø> (ø)
client/src/route/auth/AccountTabData.tsx 12.90% <ø> (ø)
client/src/route/auth/PrivateRoute.tsx 25.00% <ø> (ø)
client/src/util/envUtil.ts 35.89% <100.00%> (ø)
client/src/route/library/Library.tsx 54.54% <50.00%> (ø)
client/src/util/auth/Authentication.ts 23.68% <20.00%> (ø)

... and 60 files with indirect coverage changes

Components Coverage Δ
Website 73.52% <38.23%> (∅)
Lib Microservice ∅ <ø> (∅)

@prasadtalasila prasadtalasila merged commit c66d53c into INTO-CPS-Association:feature/distributed-demo Nov 1, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants