feat(errorboundary): add error boundaryComponent #145

Merged
andrey.dekterev merged 1 commits from error-boundary into develop 3 years ago
Owner
https://ru.reactjs.org/docs/error-boundaries.html
andrey.dekterev requested review from roman.rakov 3 years ago
andrey.dekterev requested review from margarita.polyakova 3 years ago
andrey.dekterev requested review from ruslan.khairullin 3 years ago
andrey.dekterev requested review from denis.farber 3 years ago
ruslan.khairullin reviewed 3 years ago
useEffect(() => {
open()
setTimeout(close, 5000)
Collaborator

можно добавить clearTimeout при сбросе эффекта

можно добавить clearTimeout при сбросе эффекта
andrey.dekterev marked this conversation as resolved
import {
Collaborator

import type

import type
andrey.dekterev marked this conversation as resolved
andrey.dekterev force-pushed error-boundary from fb428f9d78 to 7e89008246 3 years ago
ruslan.khairullin approved these changes 3 years ago
andrey.dekterev force-pushed error-boundary from 7e89008246 to 8b62ee20d0 3 years ago
roman.rakov reviewed 3 years ago
<ReactQueryDevtools initialIsOpen={isLocalhost} />
<AuthenticatedApp />
</QueryClientProvider>
<ErrorBoundary>

можем ли мы это перенести в компонент ?

можем ли мы это перенести в компонент <GlobalStores>?
andrey.dekterev marked this conversation as resolved
andrey.dekterev force-pushed error-boundary from 8b62ee20d0 to dd993e561c 3 years ago
andrey.dekterev force-pushed error-boundary from dd993e561c to e48b890fd7 3 years ago
andrey.dekterev merged commit e48b890fd7 into develop 3 years ago
andrey.dekterev deleted branch error-boundary 3 years ago

Reviewers

roman.rakov was requested for review 3 years ago
margarita.polyakova was requested for review 3 years ago
denis.farber was requested for review 3 years ago
ruslan.khairullin approved these changes 3 years ago
continuous-integration/drone/push Build is passing
The pull request has been merged as e48b890fd7.
Sign in to join this conversation.
No Label
No Milestone
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: frontend/spa_instat_tv#145
Loading…
There is no content yet.