americanexpress / americanexpress/react-albus

Warning that step.id is null when the Wizard is first time rendering.

Open
#29 9 comments 2 reactions 0 assignees View on GitHub
stale-issue
Dominant language
JavaScript
Stars
1.1k
Forks
88
PR merge metrics
No merged PRs in 30d

Description

Hi,

First of all great library. Does what it says and does it best.
I was trying it out and was getting warning.
`Warning: Failed prop type: The prop `step.id` is marked as required in `Steps`, but its value is `null`. in Steps (created by Wizard) in Wizard (created by Onboarding)`

Turns out the same warning was replicated in the console.log of the [official examples](http://americanexpress.io/react-albus/) from add animation onwards.

Problem:
Basically whenever I try to render Step, using Wizard's render api, this warning shows up.
The default props of Steps.id = null. When the object is rendering it throws an error.

Can you please check?

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by tracing the Steps component through the Wizard render API, then reproduce the warning in the official examples, especially from the animation example onward. Check how the required step.id prop is handled during the first render and verify that the warning no longer appears when rendering a Step through the render API.

Written by the indexing model from the issue text.

Assessment

Tech stack
react
Domain
frontend
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.