Skip to content

Commit

Permalink
Remove TODO from code
Browse files Browse the repository at this point in the history
  • Loading branch information
arkadiuszbachorski committed Jul 4, 2024
1 parent 7b8b6ce commit 5d2a5ab
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion modules/firebase/AuthProvider.ts
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,6 @@ export const AuthProvider = () => {
useRegisterAuthServiceWorker(firebaseConfig)

Check warning on line 23 in modules/firebase/AuthProvider.ts

View check run for this annotation

Codecov / codecov/patch

modules/firebase/AuthProvider.ts#L23

Added line #L23 was not covered by tests

useEffect(() => {
// TODO: This is not ideal, results with double redirect. To investigate
const isSignIn = window.location.pathname === routes.signIn

Check warning on line 26 in modules/firebase/AuthProvider.ts

View check run for this annotation

Codecov / codecov/patch

modules/firebase/AuthProvider.ts#L25-L26

Added lines #L25 - L26 were not covered by tests
if (isSignIn && user) {
window.location.assign(routes.home)

Check warning on line 28 in modules/firebase/AuthProvider.ts

View check run for this annotation

Codecov / codecov/patch

modules/firebase/AuthProvider.ts#L28

Added line #L28 was not covered by tests
Expand Down

0 comments on commit 5d2a5ab

Please sign in to comment.