FE-836: Tile + wire up the network graph component into hash-frontend - #9318
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
3 Skipped Deployments
|
PR SummaryMedium Risk Overview Adds Reviewed by Cursor Bugbot for commit a01be13. Bugbot is set up for automated code reviews on this repo. Configure here. |
91108b1 to
290d366
Compare
290d366 to
88514fa
Compare
5d641b5 to
0221332
Compare
|
|
||
| {chips.map((chip) => ( | ||
| <TypeChip key={chip.label} chip={chip} /> | ||
| ))} |
There was a problem hiding this comment.
Type chips use colliding React keys
Low Severity
Type chips are keyed only by chip.label. An entity with two types that share a title (different versions, or distinct types with the same name) gets duplicate keys, so React can reuse the wrong chip and show a stale colour or icon when the selection updates.
Reviewed by Cursor Bugbot for commit 0221332. Configure here.
eb2d762 to
22d8ec1
Compare
22d8ec1 to
ed6442b
Compare
ed6442b to
6b7d213
Compare
6b7d213 to
57bf52c
Compare
57bf52c to
a01be13
Compare


🌟 What is the purpose of this PR?
Wires up the NetworkGraph component defined in ds-components to the new atlas service. Handles parsing the atlas wire format, tile geometry, caching + prefetching + introduces a few changes to the filter bar to support the new graph view.
Pre-Merge Checklist 🚀
🚢 Has this modified a publishable library?
This PR:
📜 Does this require a change to the docs?
The changes in this PR: