Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

MCG Takehome implementation #1511

Closed
wants to merge 31 commits into from
Closed

MCG Takehome implementation #1511

wants to merge 31 commits into from

Conversation

rbuels
Copy link

@rbuels rbuels commented Feb 24, 2025 •
edited
Loading

No description provided.

rbuels and others added 30 commits February 23, 2025 17:37
@rbuels
initial documentation commit
2d3957b
@rbuels
refactor models.py into a multi-file package
286c434
@rbuels
add first draft of models for attachments and patients
f1582c1
@rbuels
feat: Add patients routes to backend API
6c0bdb7
@rbuels
feat: Add CRUD routes for patients with authorization checks
e8010e2
@rbuels
refactor: Update patients route to use Patient model instead of Item
9ec3e0b
@rbuels
test: Add comprehensive test suite for patient CRUD routes
9ead27f
@rbuels
refactor: Convert test_patients.py from async to sync tests
b3d1ccb
@rbuels
add .aider* files to gitignore
67dd823
@rbuels
register patients route
deacf0d
@rbuels
some improvements to patient and attachment models
2234f3b
@rbuels
feat: Add CRUD routes and model for Attachments
b3e3b3b
@rbuels
test: Add comprehensive tests for attachment CRUD routes
0de656b
@rbuels
get the (pretty good!) generated attachment tests passing
1472bc0
@rbuels
fix patient and attachment test data sticking around after tests run
2d24b5e
@rbuels
feat: Add GIN index on patient medical_history column for improved te…
50dae3e 
…xt search performance
@rbuels
test: Add test for patient medical history text search
b4f3a56
@rbuels
add history_text query param to GET .../patients, get its test passing
f83332c
@rbuels
test: Add tests for name_exact, name_text, and attachment MIME type f…
697315b 
…ilters
@rbuels
add more filters to .../patients route and get tests passing
fa1bb5c
@rbuels
feat: Add storage_path field to Attachment model for S3 file paths
6e64968
@rbuels
add dep on boto3 for AWS interactions
2b39d31
@rbuels
fix multiple alembic heads
8aa9d8d
@rbuels
add config and env variables for AWS credentials
0b3213d
@rbuels
first draft of attachment presigned url generation, delete routes for…
63220cc 
… updating and deleting attachments
@rbuels
fix alembic not detecting the medical history fulltext index
31db2e0
@rbuels
URL presigning starting to work
e74e709
@rbuels
test: Add test for read_attachment_content route with presigned URL r…
40cc190 
…edirect
@rbuels
get ai-written tests passing
a9ed2a9
@rbuels
remove extraneous things from the root README
7e8b778
@rbuels rbuels closed this Feb 24, 2025
@rbuels rbuels deleted the mcg-takehome branch February 24, 2025 06:17
@rbuels rbuels restored the mcg-takehome branch February 24, 2025 06:18
@rbuels
Copy link
Author

rbuels commented Feb 24, 2025

Gah sorry opened this by mistake. Forked this to do an exam project and meant to put the PR in my fork.

@rbuels rbuels deleted the mcg-takehome branch February 24, 2025 06:21
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.
1 participant
@rbuels