Skip to content

add checkbox to create follow up form#618

Open
mabiede wants to merge 5 commits into
mainfrom
bug/2435-enroll-from-main
Open

add checkbox to create follow up form#618
mabiede wants to merge 5 commits into
mainfrom
bug/2435-enroll-from-main

Conversation

@mabiede
Copy link
Copy Markdown
Contributor

@mabiede mabiede commented Jun 4, 2026

No description provided.

@mabiede mabiede self-assigned this Jun 4, 2026
Copilot AI review requested due to automatic review settings June 4, 2026 04:53
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds an admin UI control and backend handling to optionally enroll participants from a main session when creating a follow-up session.

Changes:

  • Add an EnrollFromMain checkbox to the follow-up session creation form.
  • Parse and act on the new EnrollFromMain boolean in the follow-up creation handler by enrolling existing main-session participants into the new follow-up session.
  • Extend Pool_message.Field.t and EN/DE locale mappings to support the new field label.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
pool/web/view/page/page_admin_session.ml Renders the new checkbox on the follow-up creation form.
pool/web/handler/admin_session.ml Reads the new checkbox value and conditionally creates enrollment events for follow-up assignments.
pool/pool_message/field.mli Adds EnrollFromMain to the Field.t variant type.
pool/pool_message/field.ml Adds string encoding/printing for the new EnrollFromMain field.
pool/app/pool_common/locales/locales_en.ml Adds EN label text for EnrollFromMain.
pool/app/pool_common/locales/locales_de.ml Adds DE label text for EnrollFromMain and updates the HideIgnored translation.

Comment thread pool/web/view/page/page_admin_session.ml Outdated
Comment thread pool/web/handler/admin_session.ml
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Comment thread pool/web/view/page/page_admin_session.ml
Comment thread pool/web/handler/admin_session.ml Outdated
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Comment thread pool/web/handler/admin_session.ml
Comment thread pool/web/handler/admin_session.ml
@mabiede mabiede requested a review from PossessedC0bra June 4, 2026 07:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants