Skip to content

Peer Instruction Migration to FastAPI#1209

Draft
sethbern wants to merge 3 commits intoRunestoneInteractive:mainfrom
sethbern:peer-fastapi
Draft

Peer Instruction Migration to FastAPI#1209
sethbern wants to merge 3 commits intoRunestoneInteractive:mainfrom
sethbern:peer-fastapi

Conversation

@sethbern
Copy link
Copy Markdown
Contributor

@sethbern sethbern commented May 7, 2026

Hey @bnmnetp, I have done a first pass at moving peer instruction over to FastAPI. For now I am focusing on sync PI to get things working end-to-end before touching async.

Sync PI routes are in, Jinja2 templates ported from the old web2py views, and nav links fixed across _base.html, menu.html, and the Sphinx book theme so everything points to /assignment/peer/.... Async stubs are there but not wired up yet. I've tested locally and seems to be working ok.

A couple things I wanted your eyes on before I keep going:

  1. LTI score sending, I am not super familiar with this part of the codebase so I followed the existing pattern. Does the approach look right?
  2. Overall structure good to go before I move on to async or is there anything else that seems wrong?

@bnmnetp
Copy link
Copy Markdown
Member

bnmnetp commented May 7, 2026

Thanks @sethbern . It looks like you found and cleaned up a number of things in my previous work on porting. The overall structure looks good so keep going I say.

I'll take a closer look at the LTI stuff. I'm just back from two weeks of travel, so I've got a number of other things to take care of before I'm fully back at my desk.

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