Skip to content

Support Lead TA course role#402

Open
alxstx wants to merge 1 commit intoberkeley-cdss:mainfrom
alxstx:issue-348-lead-ta-role
Open

Support Lead TA course role#402
alxstx wants to merge 1 commit intoberkeley-cdss:mainfrom
alxstx:issue-348-lead-ta-role

Conversation

@alxstx
Copy link
Copy Markdown
Contributor

@alxstx alxstx commented Apr 23, 2026

Changes
Added Lead TA as a supported staff/course-admin role across course authorization and dashboards
Added Canvas custom role mapping so internal leadta sync uses enrollment_role=Lead TA while built-in roles keep enrollment_type[]
Updated course import filtering to recognize Canvas Lead TA enrollments as staff courses
Updated course and enrollment tables to display Lead TA with a readable label
Added regression tests covering Lead TA role support, import behavior, enrollment sync, and Canvas query parameters

Testing
Run bundle exec rspec spec/models/user_to_course_spec.rb spec/models/course_spec.rb spec/controllers/courses_controller_spec.rb spec/facades/canvas_facade_spec.rb for Lead TA role tests
Run the full rspec test suite to verify no regressions
Run bundle exec rubocop app/models/user_to_course.rb app/models/course.rb app/controllers/courses_controller.rb app/facades/canvas_facade.rb spec/models/user_to_course_spec.rb spec/models/course_spec.rb spec/controllers/courses_controller_spec.rb spec/facades/canvas_facade_spec.rb to verify changed Ruby files

Documentation
No documentation needed

Checklist

  • Name of branch corresponds to story
  • 80%+ test coverage with all tests passing

@superconductor-for-github
Copy link
Copy Markdown

superconductor-for-github Bot commented Apr 23, 2026

@alxstxSuperconductor finishedView implementation | App Preview


You're out of extra usage on your Claude plan. You can wait until 12:10am (UTC), or add more extra usage at https://claude.ai/settings/billing

@alxstx
Copy link
Copy Markdown
Contributor Author

alxstx commented Apr 23, 2026

Closing because this PR should be opened against cs169/flextensions first.

@alxstx alxstx closed this Apr 23, 2026
@alxstx
Copy link
Copy Markdown
Contributor Author

alxstx commented Apr 26, 2026

Reopening — corresponding upstream PR is now open at cs169#350.

@alxstx alxstx reopened this Apr 26, 2026
@alxstx alxstx force-pushed the issue-348-lead-ta-role branch from 43dfef3 to 2f51943 Compare April 26, 2026 20:51
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.

1 participant