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

add catches to endpoint calls #2225

Closed
wants to merge 5 commits into from
Closed

Conversation

CollinBeczak
Copy link
Collaborator

Adding catches to a majority of the endpoint requests and making those requests async.

@codecov-commenter
Copy link

codecov-commenter commented Jan 7, 2024

Codecov Report

Attention: 442 lines in your changes are missing coverage. Please review.

Comparison is base (0ab7543) 23.72% compared to head (2188068) 23.62%.
Report is 1 commits behind head on main.

Files Patch % Lines
src/services/User/User.js 2.43% 112 Missing and 8 partials ⚠️
src/services/Task/Task.js 4.31% 108 Missing and 3 partials ⚠️
src/services/Challenge/Challenge.js 11.76% 59 Missing and 1 partial ⚠️
src/services/Task/TaskReview/TaskReview.js 3.12% 31 Missing ⚠️
src/services/Project/Project.js 7.40% 25 Missing ⚠️
src/services/Team/Team.js 7.69% 12 Missing ⚠️
src/services/Leaderboard/Leaderboard.js 38.88% 9 Missing and 2 partials ⚠️
src/services/VirtualChallenge/VirtualChallenge.js 0.00% 10 Missing ⚠️
src/components/HOCs/WithChallenge/WithChallenge.js 0.00% 5 Missing and 1 partial ⚠️
src/services/SuperAdmin/SuperAdminChallenges.js 0.00% 5 Missing ⚠️
... and 18 more

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2225      +/-   ##
==========================================
- Coverage   23.72%   23.62%   -0.10%     
==========================================
  Files         648      648              
  Lines       22344    22455     +111     
  Branches     6896     6897       +1     
==========================================
+ Hits         5300     5304       +4     
- Misses      14253    14359     +106     
- Partials     2791     2792       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@CollinBeczak CollinBeczak changed the title Catches to endpoint calls add catches to endpoint calls Jan 7, 2024
@CollinBeczak CollinBeczak deleted the catchesToEndpointCalls branch January 15, 2024 18:08
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