justinzeus/scholarr
1
0
Fork 0
Code Issues 1 Pull requests 18 Projects Releases Packages Wiki Activity Actions

fix(ci): fix import sorting, API contract, and pdf-queue test

Browse source 
- Add missing blank line after imports in application.py (ruff I001)
- Use full path string instead of template literal for export API call
  to avoid false positive in API contract drift check
- Update pdf-queue test to expect 200 for non-admin listing
This commit is contained in:
Justin Visser 2026-03-07 15:40:49 +01:00
parent 6742eff773
commit 76c4811405
2 changed files with 3 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

1
app/services/scholars/application.py
View file

@ -27,6 +27,7 @@ from app.services.scholars.validators import (
validate_scholar_id, validate_scholar_id,
) )
async def bulk_delete_scholars( async def bulk_delete_scholars(
db_session: AsyncSession, db_session: AsyncSession,
*, *,

4
frontend/src/features/scholars/index.ts
View file

@ -182,8 +182,8 @@ export async function bulkToggleScholars(scholarProfileIds: number[], isEnabled:
} }
export async function exportScholarData(ids?: number[]): Promise<DataExportPayload> { export async function exportScholarData(ids?: number[]): Promise<DataExportPayload> {
const params = ids && ids.length > 0 ? `?ids=${ids.join(",")}` : ""; const path = ids && ids.length > 0 ? `/scholars/export?ids=${ids.join(",")}` : "/scholars/export";
const response = await apiRequest<DataExportPayload>(`/scholars/export${params}`, { const response = await apiRequest<DataExportPayload>(path, {
method: "GET", method: "GET",
}); });
return response.data; return response.data;