fix ids of labels in invite request form in admin

[categulario]

I noticed while using the admin interface that clicking on the "Set a question for invite requests" label didn't toggle the checkbox even though the input is inside the label. Turns out the reason is the for="" attribute that is not properly set.

[joachimesque]

Looks good, but it could also work without the for attribute: the <input> element is wrapped inside the <label>.

[categulario]

most certainly. El mar., 1 de agosto de 2023 03:24, Joachim ***@***.***> escribió:

…

***@***.**** approved this pull request. Looks good, but it could also work without the for attribute: the <input> element is wrapped inside the <label>. — Reply to this email directly, view it on GitHub <#2912 (review)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAGBBZDKXPFBM5WZDPTJDSDXTDDMNANCNFSM6AAAAAA2SJSBVM> . You are receiving this because you authored the thread.Message ID: ***@***.***>

[categulario]

Would you prefer if I do it that way? it is probable better in the long run to not need ids to match.

[joachimesque]

The simplest code is the better in my opinion. If we don’t need the id, there’s less risk of error if the Form has to change :)

…

Le 1 août 2023 à 18:50, Abraham Toriz Cruz ***@***.***> a écrit : Would you prefer if I do it that way? it is probable better in the long run to not need ids to match. — Reply to this email directly, [view it on GitHub](#2912 (comment)), or [unsubscribe](https://github.com/notifications/unsubscribe-auth/AANS2TSMMB4P2YRUSA5LJGDXTEXU3ANCNFSM6AAAAAA2SJSBVM). You are receiving this because you commented.Message ID: ***@***.***>

[categulario]

I updated the code based on this conversation

[categulario]

Let me know if mor changes are necessary, or if a rebase is desirable

[mouse-reeve]

Perfect, thank you so much!