This PR continues to make progress on replacing the modal mixin with modalData() (part of #676). This PR uses modalData() in components that contain an async modal component. The modal mixin is awkward to use with async modal components.
What has been done to verify that this works as intended?
Tests continue to pass.
Before submitting this PR, please make sure you have:
[x] run npm run test and npm run lint and confirmed all checks still pass OR confirm CircleCI build passes
[x] verified that any code or assets from external sources are properly credited in comments or that everything is internally sourced
This PR continues to make progress on replacing the
modalmixin withmodalData()(part of #676). This PR usesmodalData()in components that contain an async modal component. Themodalmixin is awkward to use with async modal components.What has been done to verify that this works as intended?
Tests continue to pass.
Before submitting this PR, please make sure you have:
npm run testandnpm run lintand confirmed all checks still pass OR confirm CircleCI build passes