This PR makes progress on #676. It replaces the modal mixin with the modalData() function. The modal mixin is our last mixin, so once this PR is merged, we will officially have no more mixins!
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 makes progress on #676. It replaces the
modal
mixin with themodalData()
function. Themodal
mixin is our last mixin, so once this PR is merged, we will officially have no more mixins!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 test
andnpm run lint
and confirmed all checks still pass OR confirm CircleCI build passes