Make breadcrumbs props less leaky and more tree/linked-list-like (!12796) · Merge requests · Administrator / sourcegraph

Administrator requested to merge breadcrumb-linked-list into marek/repo-header-contributions-prop Aug 06, 2020

Created by: felixfbecker

This is an attempt at https://github.com/sourcegraph/sourcegraph/pull/12536#issuecomment-667259404

Unfortunately I am running into these errors:

Warning: Cannot update a component from inside the function body of a different component.

Maximum update depth exceeded. This can happen when a component repeatedly calls setState inside componentWillUpdate or componentDidUpdate. React limits the number of nested updates to prevent infinite loops.

I'm not sure how to prevent them (if this approach is inherently flawed, or whether I overlooked something and made a mistake somewhere)

Make breadcrumbs props less leaky and more tree/linked-list-like

Merge request reports