Migration nav item should not have any sub items & should show what's currently in Overview

[AllenBW]

requires #3683 before this can get in...

closes #161 so lets WIP it till then... but please do feedback ah meeeeee so this can be made as perfect as possible for as soon as #3683 makes a move

looks like #3683 made it in! this is g2g

so whats this look like?

updated breadcrumb

but gotta a question... no more questions!

@serenamarie125 @vconzola yah want the breadcrumb collapsed as well?

@priley86 that other component we dont look at InfrastructureMappings, yah want it cleaned up in this work as well?

[AllenBW]

@priley86 are we merging into sprint6 or master?

[serenamarie125]

@AllenBW this is looking good 👏

@vconzola I'm wondering about the breadcrumb. Currently as Zika mentioned previously CloudForms isn't utilizing breadcrumbs consistently. Do you think we should keep it? In this case, if we keep it, none of the "crumbs" should be links, they should just show the trail ( since there are no navigatable elements )

[priley86]

@AllenBW we are targeting sprint6 branch. Please feel free to update/remove react router routes accordingly. Thanks!!

[AllenBW]

@serenamarie125 @priley86 this is g2g whenever ya'll are!

[priley86]

Thanks a lot @AllenBW - this was more than expected ;)

Code looks fine. Just waiting on breadcrumb confirmation @vconzola @serenamarie125 .

[priley86]

Hey @AllenBW - can we go ahead and set the Breadcrumb to Compute > Migrate ? I believe @vconzola would prefer this. It would be even nicer if the breadcrumbs had a single arrow like PF Core. I believe we still need to update PF in PF-React to latest if you want to try this...

[AllenBW]

@priley86 on it!

[AllenBW]

@priley86 @vconzola updated breadcrumb as requested

[priley86]

thanks @AllenBW 🥇

i will go ahead and merge... let's target the PF upgrade in PF-React next if you are up to investigating that...hopefully it resolves the Breadcrumbs design (this was mentioned during the last community meeting i believe).

[AllenBW]

@priley86... yeah can up the dep in pfr, thoughts on updating the whole slew of em vs just the one? (leaning update all we can cuz... 🆙 deps are 😀 deps)