Matt Rakow
Matt Rakow
Not entirely sure what's causing the test failure, needs some investigation.
> Currently running npm run webpack in the root fails due to issues with css in prosemirror: these issues are unchanged by my changes, but leaves me without a great...
Looks like the demo still errors at runtime using 1.2.1 but at least packing succeeds. Probably needs a little more attention.
> I am unfortunately not familiar with the APIs here. My first impression is that such a property about container doesn't belong on client props. I see AzureContainerVersion nearby. Is...
> One more thing: If the createAsEphemeral flag is added to createFluidContainer, I think we need to update the AzureClient.ts file to pass the flag to all references of createFluidContainer...
> I would argue that the time to do that sort of investigation is when updating our min requirements, rather than when updating dependencies in accordance with those requirements. I...
Marking this ready for review - I had a chance to experiment with this and it worked as expected in a variety of scenarios. Main downside is that when attempting...
I know some of these but not all: 1. Everything related to audience seems expected (#20703) 2. Everything related to LoaderHeader seems expected (#20793) 3. I think the stuff around...
For visibility, we're currently looking at resolving test pipeline issues before merging this scope change.
@Abe27342 I believe @pk-pranshu is going to drive this discussion? I never got much response to my comment in [AB#5699](https://dev.azure.com/fluidframework/235294da-091d-4c29-84fc-cdfc3d90890b/_workitems/edit/5699), so that might be a good place to weigh in...