Skip to content

feat: SIS deep-link from student detail view (#78)#95

Merged
William-Hill merged 15 commits intomainfrom
feature/sis-deep-link
Apr 2, 2026
Merged

feat: SIS deep-link from student detail view (#78)#95
William-Hill merged 15 commits intomainfrom
feature/sis-deep-link

Conversation

@William-Hill
Copy link
Copy Markdown
Collaborator

@William-Hill William-Hill commented Apr 2, 2026 •
edited
Loading

Summary

  • Add FERPA-compliant "Open in SIS" button to the student detail page that constructs a deep-link URL to the institution's SIS server-side — the SIS student ID never reaches the browser
  • New guid_sis_map table + seed script with 20 demo mappings for POC/demo purposes
  • New API route GET /api/students/[guid]/sis-link with role gating (admin/advisor/ir), server-side URL construction, and fire-and-forget audit logging
  • Button shows three states: clickable (mapping exists), disabled with tooltip (no mapping), or hidden (unauthorized/feature disabled)
  • Feature controlled by SIS_BASE_URL env var — leave blank to disable entirely

Test plan

  • Set SIS_BASE_URL=https://sis-demo.example.com/students and SIS_ID_PARAM=id in .env.local
  • Run python -m operations.seed_guid_sis_map to create table and seed demo data
  • Navigate to a seeded student — verify "Open in SIS" button is clickable and opens demo URL in new tab
  • Navigate to a non-seeded student — verify button is disabled with tooltip
  • Log in as faculty/leadership role — verify button is hidden
  • Remove SIS_BASE_URL from env, restart — verify button is hidden entirely
  • Check logs/query-history.jsonl for sis_link_accessed audit entries (GUID + role only, no SIS ID)

Closes #78

William-Hill and others added 15 commits February 24, 2026 13:08
@William-Hill
docs: design doc for self-service data upload (issue #86)
075b3f5
@William-Hill
docs: implementation plan for self-service data upload (issue #86)
184202e
@William-Hill
docs: design spec for config-driven distillation pipeline
cb20f2d 
Per-school fine-tuning pipeline to replace OpenAI dependency for
explanation and summarization endpoints with locally-served Qwen 3.5
models via MLX and Ollama.
@William-Hill
docs: implementation plan for config-driven distillation pipeline
ae971e4
@William-Hill
docs: add design spec for SIS deep-link feature (#78)
29af105
@William-Hill
docs: add implementation plan for SIS deep-link feature (#78)
8b89b7f
@William-Hill
feat(#78): add guid_sis_map seed script for SIS deep-link POC
7e2bd03
@William-Hill
fix(#78): use get_connection helper and add error handling to seed sc…
2e7b9e0 
…ript
@William-Hill
feat(#78): add SIS deep-link API route with audit logging
b70cdda
@William-Hill
fix(#78): use canAccess pattern and isolate audit log writes
c06d4f8
@William-Hill
feat(#78): add Open in SIS button to student detail page
f7cd37e
@William-Hill
fix(#78): handle loading state, null URL edge case, and button styling
30a617f
@William-Hill
fix(#78): add AbortController to SIS link fetch to prevent stale updates
a9d17ab
@William-Hill
refactor(#78): simplify route, consolidate button markup, use execute…
abc8628 
…many
@William-Hill
fix(#78): use URL API for robust query param handling
c76fde7
@William-Hill William-Hill merged commit b9cbe63 into main Apr 2, 2026
3 checks passed
@William-Hill William-Hill deleted the feature/sis-deep-link branch April 2, 2026 17:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

feat: SIS deep-link from student detail view (FERPA-compliant identity resolution)

1 participant

@William-Hill