Skip to content

Commit

Permalink
fix: DPA-2752 Make roles option field for ADS (#44)
Browse files Browse the repository at this point in the history
* fix: DPA-2752 Make roles option field for ADS

* add files
  • Loading branch information
jatinyadav-cc authored Aug 26, 2024
1 parent 3054e80 commit 674edfa
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 12 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ data:
connectorSubtype: api
connectorType: source
definitionId: 694710ca-8933-4661-8db9-c38cb60411b0
dockerImageTag: 0.2.5
dockerImageTag: 0.2.6
dockerRepository: datapipes-airbyte/source-brightspace
githubIssueLabel: source-brightspace
icon: brightspace.svg
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -278,7 +278,6 @@ connectionSpecification:
required:
- org_unit_id
- start_date
- roles
properties:
dataset_type:
type: string
Expand Down Expand Up @@ -314,7 +313,6 @@ connectionSpecification:
required:
- org_unit_id
- start_date
- roles
properties:
dataset_type:
type: string
Expand Down Expand Up @@ -379,7 +377,6 @@ connectionSpecification:
required:
- org_unit_id
- start_date
- roles
properties:
dataset_type:
type: string
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -574,13 +574,15 @@ def create_export_job(self) -> BSExportJob:
{
"name": "endDate",
"value": self.end_date
},
{
"name": "roles",
"value": ",".join(map(str, self.roles))
}

]
}
if self.roles is not None:
payload['Filters'].append({
"name": "roles",
"value": ",".join(map(str, self.roles))
})
return self.bs_api.create_export_job(payload=payload)


Expand Down Expand Up @@ -655,13 +657,15 @@ def create_export_job(self) -> BSExportJob:
{
"name": "endDate",
"value": self.end_date
},
{
"name": "roles",
"value": ",".join(map(str, self.roles))
}
]
}
if self.roles is not None:
payload['Filters'].append({
"name": "roles",
"value": ",".join(map(str, self.roles))
})

return self.bs_api.create_export_job(payload=payload)


Expand Down

0 comments on commit 674edfa

Please sign in to comment.