Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request introduces an error boundary component to the
frontend/webapp
to handle errors gracefully and updates several dependencies. The most important changes include the addition of theAppErrorBoundary
component, updates to theRootLayout
to use the new error boundary, and modifications to handle nullable conditions in various types.Error handling improvements:
frontend/webapp/app/layout.tsx
: AddedAppErrorBoundary
component to wrap the main layout, ensuring that errors are caught and displayed in a user-friendly manner. [1] [2] [3]frontend/webapp/components/app-error-boundary/index.tsx
: Created theAppErrorBoundary
component usingreact-error-boundary
to provide a fallback UI in case of errors.frontend/webapp/components/index.ts
: Exported the newly createdAppErrorBoundary
component.Dependency updates:
frontend/webapp/package.json
: Updated several dependencies, including@apollo/client
,@odigos/ui-components
,@odigos/ui-utils
, and addedreact-error-boundary
.Handling nullable conditions:
frontend/webapp/types/actions.ts
: UpdatedActionData
interface to allowconditions
to benull
.frontend/webapp/types/destinations.ts
: UpdatedDestination
andActualDestination
interfaces to allowconditions
to benull
. [1] [2]frontend/webapp/types/sources.ts
: UpdatedK8sActualSource
interface to allowconditions
to benull
.