✨ Fields should be required if specific location is selected

[Jerryescandon]

🕵️ Details

On the pool edit page (/admin/pools/:pool-id/edit), the fields that appear after Location (Specific Location (English), Specific Location (French)) should be made required if the Specific location (Specify below) radio is selected.

📸 Screenshot

✅ Acceptance Criteria

A set of assumptions which, when tested, verify that the bug was addressed.

• [ ] the fields that appear after Location (Specific Location (English), Specific Location (French)) should be made required if the Specific location (Specify below) radio is selected

[petertgiles]

This one's a bit sticky. All the fields on the edit pool page are marked optional because the user is working on a draft. They need to be able to edit other fields (like language requirement or publishing group) without being required to fill all the other fields before updating the draft.

I agree that this looks weird but I think it's part of a bigger conversation about the design of this page. Related: #3974.

[Jerryescandon]

We can probably icebox this for now. I have a bigger issue with "Drafts" as a whole, I think we should create a more robust drafting system, but it might be a bigger issue for a later epic.

[mnigh]

Related #5626.

[mnigh]

@esizer could this be done similar to https://github.com/GCTC-NTGC/gc-digital-talent/issues/5535#issuecomment-1854192803?

[esizer]

@esizer could this be done similar to #5535 (comment)?

Yup! I'm pretty sure react-hook-form only validates visible fields 👍

[vd1992]

Appears resolved with the pool editor redesign?

[mnigh]

Appears resolved with the pool editor redesign?

No, these fields are still not marked as required.