populationgenomics / seqr

web-based analysis tool for rare disease genomics
GNU Affero General Public License v3.0
3 stars 1 forks source link

Change project guid arg to project in sa sync function #224

Closed EddieLF closed 1 month ago

EddieLF commented 1 month ago

Hotfix for bug introduced in #222, I put project_guid as the project arg in the validate_fam_file_records function, but really it needs the Numerical project ID.

With project_guid in this place, there is a different error:

ValueError: Field 'id' expected a number but got <PROJECT_GUID_STRING>

Luckily the numerical project ID is fetched a few lines above: project = get_project_and_check_pm_permissions(project_guid, request.user)