Evan Mattson

Thanks @hussain-t – this is almost there, just a few points. > Move the following REST endpoints-related helper methods: > * get_module_dependencies, > * get_module_schema, > * prepare_module_data_for_response > *...

Thanks @hussain-t IB ✅

Maintainers have been notified about this but no update yet.

> The AC suggests rendering the `` but making it "Visually Hidden". Does this mean wrapping the entire component within the `` component? I feel this bug can be resolved...

@techanvil I've tweaked the language to hopefully avoid misinterpretation that the banner should be displayed/visible when the feature flag is enabled as it has its own logic for determining that...

Thanks @eclarke1 – apologies for the delay here. I hadn't forgotten about it 👍 @kuasha420, nice work here – I like the idea to model this after `Notices`, that should...

IB ✅

@kuasha420 let's opt to go with the alternate placement, putting it under the menu instead. I'm sure core probably has its own pointers that could end up below the admin...

Reopening this one to consider an amendment as raised by @wpdarren and @jamesozzie. While this currently works as intended, in retrospect, it was not ideal to restart the setup flow...

Great, thanks @techanvil and @wpdarren !