Skip to content

Fix synchronous PI based on testing/feedback#1227

Closed
sethbern wants to merge 5 commits into
RunestoneInteractive:mainfrom
sethbern:sync-pi-feedback
Closed

Fix synchronous PI based on testing/feedback#1227
sethbern wants to merge 5 commits into
RunestoneInteractive:mainfrom
sethbern:sync-pi-feedback

Conversation

@sethbern
Copy link
Copy Markdown
Contributor

@sethbern sethbern commented Jun 4, 2026

Fixed several sync PI issues from testing feedback:

  • Show real student names in the verbal group select dropdown instead of anonymous names
  • Fixed premature "discuss" message after vote 1 stops
  • Added line numbers to code blocks displayed in PI questions (does not touch the actual questions, just adds them visually during a PI session based on instructor feedback that it makes discussion easier)
  • Improved group select placeholder text

Also, some PI pages still seem to route to web2py instead of FastAPI. I think it's because nginx serves /staticAssets/ from the web2py static directory, but if it's something else on my end let me know and I can fix it.

@sethbern sethbern requested a review from bnmnetp as a code owner June 4, 2026 15:46
@sethbern sethbern closed this Jun 4, 2026
@sethbern sethbern deleted the sync-pi-feedback branch June 4, 2026 15:52
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