Update to use v-model
directive in commit change modal
Everyone can contribute. Help move this issue forward while earning points, leveling up and collecting rewards.
The following discussion from !174093 (merged) should be addressed:
-
@pslaughter started a discussion: suggestion (non-blocking): In another MR, it'd be nice to refactor these modals to use the
v-model
directive for managing visibility instead of calling child component methods.According to our Pajamas docs:
You can use the v-model directive to control the modal’s visibility. The v-model directive interfaces with the visible property and the @change event.
...
Note: It is recommended to use thethis.$bvModal.show()
andthis.$bvModal.hide()
methods (mentioned in the previous section) instead of using$ref
methods.
Edited by 🤖 GitLab Bot 🤖