Ben Lambert
Ben Lambert
Think we can close this at this point. Should be covered by the `ui:field: Secret` instead.
Hahah sorry, I forgot about his review still blocking this, I thought it was in already!
Does it make sense to do this by default? What's the value behind making this configurable? 😁
> Giving users flexibility in how the UI looks is a value-add IMO I'd argue that we already do that by allowing people to provide their own `ReviewStep` component. If...
Nice thanks! :pray:
superseded by https://github.com/backstage/backstage/pull/25385
Hmm, thinking that if this is a change that we're going to want to bring in, it might make sense to batch some of these breaking changes together with NBS...
@camilaibs what do you think is best for these docs updates? With the NBS getting to 1.x very soon, does it make sense to just remove the old documentation entirely,...
@OrkoHunter can I get a review here, or tag someone that can review this? :pray:
Thanks! :tada: