Closed PhotoNomad0 closed 9 months ago
Attention: 7 lines
in your changes are missing coverage. Please review.
Comparison is base (
c6612ba
) 51.64% compared to head (a5aa5bc
) 50.56%.
Files | Patch % | Lines |
---|---|---|
src/context/AuthContext.js | 0.00% | 7 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
LGTM, seems to work as it should. If you don't close the app before switching, come back, and make an edit, it throws an error but that is to be expected.
@kintsoogi Yes, there is a second part that will need to be done eventually to completely fix the issue: When there is 401 error on edit - the popup should tell you that you are no longer logged in and give you the option of going to login screen. But that may get complicated due to the warning about unsaved changes.
Describe what your pull request addresses
Test Instructions