Skip to content

fix(enrollment): guide learners who submit the wrong template out of the trap - #240

Merged
accesswatch merged 1 commit into
mainfrom
fix/enrollment-wrong-template-traps
Jul 4, 2026
Merged

accesswatch merged 1 commit into
mainfrom
fix/enrollment-wrong-template-traps

Conversation

@accesswatch

Copy link
Copy Markdown
Collaborator

A learner clicked the enrollment link but submitted the Start Here roadmap
template (issue #237): the [ENROLL] title from the URL survived, the body
had the readiness checklist but no Full Name or Email Address fields, and
the needs-info bot told them to 'edit this issue and add the missing
information' without saying how. They also got the Gandalf first-issue
greeting mid-signup, which read as noise.

  • registration.yml: the needs-info comment now explains the wrong-template
    cause, gives a copy-paste markdown block for the missing fields, spells
    out how to edit an issue body, and links the real enrollment form.
  • learning-room-pr-bot.yml: the first-issue greeting skips [ENROLL] and
    [REGISTER] intake issues.
  • start-here-roadmap.yml (main repo chooser only; learning-room copy
    unchanged): renamed to 'Course Roadmap Check-In (already enrolled
    students)' with an explicit not-an-enrollment warning and a link to the
    enrollment form.

Co-Authored-By: Claude Fable 5 noreply@anthropic.com

…the trap

A learner clicked the enrollment link but submitted the Start Here roadmap
template (issue #237): the [ENROLL] title from the URL survived, the body
had the readiness checklist but no Full Name or Email Address fields, and
the needs-info bot told them to 'edit this issue and add the missing
information' without saying how. They also got the Gandalf first-issue
greeting mid-signup, which read as noise.

- registration.yml: the needs-info comment now explains the wrong-template
  cause, gives a copy-paste markdown block for the missing fields, spells
  out how to edit an issue body, and links the real enrollment form.
- learning-room-pr-bot.yml: the first-issue greeting skips [ENROLL] and
  [REGISTER] intake issues.
- start-here-roadmap.yml (main repo chooser only; learning-room copy
  unchanged): renamed to 'Course Roadmap Check-In (already enrolled
  students)' with an explicit not-an-enrollment warning and a link to the
  enrollment form.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@accesswatch
accesswatch requested a review from taylorarndt as a code owner July 4, 2026 17:11
Copilot AI review requested due to automatic review settings July 4, 2026 17:11
@accesswatch
accesswatch merged commit 801e5c6 into main Jul 4, 2026
4 checks passed
@accesswatch
accesswatch deleted the fix/enrollment-wrong-template-traps branch July 4, 2026 17:11
@github-actions

github-actions Bot commented Jul 4, 2026

Copy link
Copy Markdown
Contributor

Peer Review Assigned

Hi @accesswatch! Your PR has been automatically paired with @taylorarndt for peer review.

For @taylorarndt:

This is a great opportunity to practice code review skills! Here's what to look for:

Content Quality:

  • Does the change accomplish what the issue describes?
  • Is the writing clear and helpful?
  • Are there any typos or grammar issues?

Accessibility:

  • Proper heading hierarchy (H1 → H2 → H3, no skips)?
  • Descriptive link text (not "click here")?
  • Alt text on images?
  • [TODO] markers removed?

Documentation:

  • Code blocks are properly formatted?
  • Tables have headers?
  • References/links work correctly?

Review Guidelines:

  • Be kind and constructive
  • Suggest improvements, don't just point out problems
  • Ask questions if something is unclear
  • Approve when ready or request changes with explanation

Resources:


Pairing by Learning Room Grouping Engine

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR improves the enrollment/intake experience by reducing confusion when learners open the wrong issue template and by preventing unrelated automation comments from appearing during signup.

Changes:

  • Expanded the “missing required values” needs-info comment to explain the wrong-template cause and provide copy/paste fields plus editing instructions and a direct enrollment link.
  • Updated the “first issue” greeting workflow to skip [ENROLL] and [REGISTER] intake issues.
  • Renamed the “Start Here” roadmap template to clearly indicate it is for already-enrolled students and added a prominent warning with a link to the enrollment form.

Reviewed changes

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

File Description
.github/workflows/registration.yml Improves needs-info guidance when enrollment issues are missing required fields due to the wrong template.
.github/workflows/learning-room-pr-bot.yml Avoids posting the first-issue greeting on enrollment/registration intake issues.
.github/ISSUE_TEMPLATE/start-here-roadmap.yml Rebrands the roadmap template for already-enrolled learners and adds a “not enrollment” warning + enrollment link.

Comment on lines +1 to 3
name: "Course Roadmap Check-In (already enrolled students)"
description: For students who already have a learning room. Enrolling? Use "Workshop Enrollment - New Student Fast Path" instead.
title: "Start Here: Student Course Roadmap (@{username})"
Comment on lines 9 to +13
## Start Here: Student Course Roadmap

> **Not enrolled yet?** This form does not enroll you. Use the
> [Workshop Enrollment form](https://github.com/Community-Access/git-going-with-github/issues/new?template=enrollment.yml&title=%5BENROLL%5D+GIT+Going+with+GitHub)
> instead - it asks for your name and email and creates your private learning room.
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.

2 participants