1628991 - Span children are missing "key" prop

Status: RESOLVED FIXED

(Tree Management :: Perfherder, defect, P1)

Description

[Ionuț Goldan [:igoldan]]

When going to Health view locally, in development mode, the following console warning shows up:

Warning: Each child in a list should have a unique "key" prop.

Check the render method of `ItemList`. See https://fb.me/react-warning-keys for more information.
    in span (at ItemList.jsx:10)
    in ItemList (at HealthTable.jsx:39)
    in Cell (created by ReactTable)
    in div (created by TdComponent)
    in TdComponent (created by ReactTable)
    in div (created by TrComponent)
    in TrComponent (created by ReactTable)
    in div (created by TrGroupComponent)
    in TrGroupComponent (created by ReactTable)
    in div (created by Tbody)
    in Tbody (created by ReactTable)
    in div (created by TableComponent)
    in TableComponent (created by ReactTable)
    in div (created by ReactTable)
    in ReactTable (at HealthTable.jsx:73)
    in div (created by Container)
    in Container (at HealthTable.jsx:72)
    in HealthTable (at HealthTableControls.jsx:71)
    in div (created by Container)
    in Container (at HealthTableControls.jsx:64)
    in HealthTableControls (at HealthView.jsx:143)
    in div (created by Container)
    in Container (at HealthView.jsx:129)
    in ErrorBoundary (at HealthView.jsx:125)
    in HeathView (at Validation.jsx:199)
    in Validation (at App.jsx:220)
    in Route (at App.jsx:217)
    in Switch (at App.jsx:79)
    in main (at App.jsx:73)
    in Router (created by HashRouter)
    in HashRouter (at App.jsx:66)
    in App (created by HotExportedApp)
    in AppContainer (created by HotExportedApp)
    in HotExportedApp (at perfherder/index.jsx:12)

We need to fix it.

Comment 1

[SSH]

Hey,
Can I work on this bug?
Could you provide more details about the files that are needed to be modified?

Comment 2

[Ionuț Goldan [:igoldan]]

I'm sorry, but this is part of Andra's ramp up.
Andra, could you pick this up?

Comment 3

[Andra Esanu (needinfo me)]

(In reply to Ionuț Goldan [:igoldan] from comment #2)

I'm sorry, but this is part of Andra's ramp up.
Andra, could you pick this up?

Sure

Comment 4

[GitHub Bugzilla PR Linker]

Attachment: Link to GitHub pull-request: https://github.com/mozilla/treeherder/pull/6301