feat(in-575): mailings bugfix

margarita.polyakova commented 3 years ago

Collaborator

margarita.polyakova added 1 commit 3 years ago

margarita.polyakova requested review from andrey.dekterev 3 years ago

andrey.dekterev reviewed 3 years ago

src/features/AuthStore/hooks/useAuth.tsx Outdated

andrey.dekterev commented 3 years ago

Owner

мы тут используем history, давай оттуда и доставать путь и токен тоже

хотя у нас даже есть хэлпер с проверкой страниц, может его использовать, checkPage, по-моему называется

margarita.polyakova commented 3 years ago

Poster

Collaborator

checkPage здесь не подходит

andrey.dekterev marked this conversation as resolved

andrey.dekterev reviewed 3 years ago

src/features/AuthStore/hooks/useAuth.tsx Outdated

margarita.polyakova added 1 commit 3 years ago

andrey.dekterev approved these changes 3 years ago

andrey.dekterev merged commit 061b09d1c4 into develop 3 years ago

andrey.dekterev deleted branch IN-575-mailings-bug 3 years ago

andrey.dekterev approved these changes 3 years ago

continuous-integration/drone/push Build is passing

Details

The pull request has been merged as 061b09d1c4.

Sign in to join this conversation.

No Label

No Milestone

No Assignees

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: frontend/spa_instat_tv#204

     const loadedUser = await userManager.getUser()
     if (!loadedUser) {
       if (window.location.pathname.includes(PAGES.mailings)) {

         const url = new URL(window.location.href)
         const access_token = url.searchParams.get('access_token') || ''
         writeToken(access_token)

feat(in-575): mailings bugfix #204