Skip to content

Commit

Permalink
fix(report): jobids are strings
Browse files Browse the repository at this point in the history
  • Loading branch information
johngeorgewright committed May 10, 2024
1 parent ea190df commit 79b53c7
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/report/runReport.ts
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ export function ensureCorrectOrderOfReportQueryParameters(
export async function runReport(
client: ReportService.ReportServiceClient,
query: ReportService.ReportQuery,
): Promise<number> {
): Promise<string> {
const [reportJob] = await client.runReportJobAsync({
reportJob: {
reportQuery: ensureCorrectOrderOfReportQueryParameters(query),
Expand All @@ -88,7 +88,7 @@ export async function runReport(

export async function waitForReportToFinish(
client: ReportService.ReportServiceClient,
jobId: number,
jobId: string,
statusCheckInterval: number,
): Promise<void> {
let status = 'IN_PROGRESS'
Expand All @@ -110,7 +110,7 @@ export async function waitForReportToFinish(

export async function streamReportResult(
client: ReportService.ReportServiceClient,
jobId: number,
jobId: string,
exportFormat: RunAndDownloadReportOpts['exportFormat'],
): Promise<IncomingMessage> {
const [urlResult] = await client.getReportDownloadURLAsync({
Expand Down

0 comments on commit 79b53c7

Please sign in to comment.