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

Setup Modal provider with zustand #23

Merged
merged 8 commits into from
Aug 1, 2024
Merged

Setup Modal provider with zustand #23

merged 8 commits into from
Aug 1, 2024

Conversation

djnunez-aot
Copy link
Collaborator

-add modal provider to application

Copy link

codecov bot commented Aug 1, 2024

Codecov Report

Attention: Patch coverage is 15.38462% with 11 lines in your changes missing coverage. Please review.

Project coverage is 30.37%. Comparing base (1436a43) to head (86f8e16).

Files Patch % Lines
...mit-web/src/components/Shared/Modals/modalStore.ts 14.28% 5 Missing and 1 partial ⚠️
...bmit-web/src/routes/_authenticated/users/index.tsx 0.00% 3 Missing ⚠️
submit-web/src/components/App/Users/UserModal.tsx 33.33% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop      #23      +/-   ##
===========================================
+ Coverage    30.22%   30.37%   +0.14%     
===========================================
  Files           26       27       +1     
  Lines          268      270       +2     
  Branches        50       52       +2     
===========================================
+ Hits            81       82       +1     
  Misses         172      172              
- Partials        15       16       +1     
Flag Coverage Δ
submit-web 30.37% <15.38%> (+0.14%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@jadmsaadaot jadmsaadaot closed this Aug 1, 2024
@jadmsaadaot jadmsaadaot reopened this Aug 1, 2024
@jadmsaadaot jadmsaadaot merged commit 795249b into develop Aug 1, 2024
9 of 10 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