Skip to content

Add two new course types to the Course Type picklist - #1192

Open
rustynwac wants to merge 1 commit into
mainfrom
claude/triage-1188-b3gdnn
Open

Add two new course types to the Course Type picklist#1192
rustynwac wants to merge 1 commit into
mainfrom
claude/triage-1188-b3gdnn

Conversation

@rustynwac

@rustynwac rustynwac commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Description

Adds two new course types to the shared Course Type picklist:

  • Intro to Avalanches Field Course (intro-to-avalanches-field-course)
  • Level 1 + Rescue Combined (level-1-rescue-combined)

courseTypesData in src/constants/courseTypes.ts is the single source of truth for course types — the Courses courseType select, the Providers "Approved Course Types" select, the public course-type filter, the embed generator, and the provider-published email all derive their options from it. Appending the two entries there makes them available everywhere at once. The options are global to all avalanche centers (course types are not tenant-scoped), which is the intended behavior.

The new entries include label and value only — no description. The description field on CourseType is optional and is rendered nowhere (every consumer maps the data down to { label, value }), so adding description copy would be invisible dead data.

Related Issues

Closes #1188

Key Changes

  • Append two new entries to courseTypesData in src/constants/courseTypes.ts (existing six entries untouched).

How to test

  1. In the admin panel, create or edit a Course and confirm both new options are selectable as the Course Type (subject to the provider's approved-types filtering that already applies).
  2. Edit a Provider and confirm both new options are selectable under "Approved Course Types".
  3. On the public course listing, confirm the "Course Type" filter shows both new options.

Screenshots / Demo video

N/A — picklist value addition, no visual/layout change.

Migration Explanation

No migration required. The stored columns for these select values (courses.course_type and providers_course_types.value) are plain text with no enum/check constraint, so new option values are accepted without a schema change.

Future enhancements / Questions

  • CourseType.description is currently carried in the constant but rendered nowhere. If descriptions should ever be surfaced (admin field help text, public filter tooltips, etc.), that would be a separate change — and the two new types would need description copy at that point.

🤖 Generated with Claude Code

https://claude.ai/code/session_012xu99cZaGFPoaP11W3K7hB


Generated by Claude Code


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

Adds "Intro to Avalanches Field Course" and "Level 1 + Rescue Combined"
to the shared courseTypesData constant, making them selectable as a
Course's courseType and a Provider's approved course types, and visible
in the public course-type filter. Values are stored as plain text, so no
migration is required.

Closes #1188

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012xu99cZaGFPoaP11W3K7hB
@github-actions

Copy link
Copy Markdown
Contributor

Preview deployment: https://claudextriage-1188-b3gdnn.preview.avy-fx.org

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.

Add values to Courses collection Course Type picklist

2 participants