Migrate deprecatedModal to GlModal in app/assets/javascripts/boards/components/board_form.vue

app/assets/javascripts/boards/components/board_form.vue

Instructions

• Replace each instance of deprecatedModal with GlModal.
• Update spec in spec/frontend/boards/components/board_form_spec.js
• Take before/after screenshots of each button in the file, and upload them in the Merge Request description.
• Request review from a designer within your stage group. If you are outside of a stage group, ping a UX foundations member.

added to epic &3938 (closed)

added component:modal pajamasintegrate typemaintenance labels

@tauriedavis FYI the deprecated modal is still used in at least one end-to-end (QA) test via this page object using this selector.

I believe that's the last place it's used in the e2e tests (after !41029 (merged))

It would be great if that page object could be updated at the same time. Anyone in Quality can help with that if needed, myself included 😀

cc @jareko (he is leading the modal work)

Thanks @mlapierre I'll be sure to ping you for a code review 👍

assigned to @jareko

mentioned in merge request !46172 (closed)

@sstern 👋 I tried migrating this modal to gl-modal but realized quickly this is going to need a bit of refactoring. Are you or someone else in groupproject management frontend able to take this on as part of our Pajamas migration efforts?

cc @kushalpandya @psimyn

cc @donaldcook for scheduling

@jareko Yup, we'll take it on.

@psimyn @fguibert Let's do this as we're refactoring boards.

@donaldcook Awesome, appreciate it!

@donaldcook Assigning you to show this issue is accounted for. Feel free to reassign to appropriate DRI 👍.

Thanks @jareko. Actually going to unassign myself otherwise FE engineers may think I am doing the work. Someone will pick it up in %13.7 though.

unassigned @jareko

added workflowscheduling label

assigned to @annabeldunstone

Assigned myself without reading the above 😅

unassigned @annabeldunstone

changed milestone to %13.7

added devopsplan frontend groupproduct planning labels

added [deprecated] Accepting merge requests label

added sectiondev label

assigned to @donaldcook

added workflowready for development label and removed workflowscheduling label

set weight to 2

unassigned @donaldcook

assigned to @fguibert

added workflowin dev label and removed workflowready for development label

mentioned in merge request !48647 (merged)

removed [deprecated] Accepting merge requests label

added workflowin review label and removed workflowin dev label

@fguibert It looks like we may be able to close this issue since the MR is merged. Can you confirm?

@tauriedavis Absolutely! Closing.

closed

added workflowproduction label and removed workflowin review label