🐛 VigneshSankar01/BambooHR - connector/fix schema updation for custom reports stream

[Vignesh S (VigneshSankar01)]

What

• This fix aims to correct the custom reports stream in airbyte for the bambooHR marketplace connector by updating the schema under that particular stream.

How

Updated the manifest.yaml file: Adjusted the configuration to include all desired fields in the custom_reports_stream. This ensures comprehensive data retrieval from BambooHR.​

Modified the fields parameter: Changed the fields parameter in the request body to reference the configuration value, ensuring that all specified fields are included in the custom report.

Review guide

manifest.yaml: Review the updates made to the custom_reports_stream configuration, ensuring that the fields parameter correctly references the desired fields.)

User Impact

Enhanced Data Retrieval: Users will now receive comprehensive data from BambooHR, as all specified fields are included in the synchronization process.​

Improved Data Integrity: By retrieving all desired columns, data integrity is maintained, facilitating better analysis and reporting.​

No Negative Side Effects: There are no anticipated negative side effects from this update. The changes are designed to enhance the connector's functionality without disrupting existing workflows.

Can this PR be safely reverted and rolled back?

• YES 💚

[vercel]

Vignesh S (@VigneshSankar01) is attempting to deploy a commit to the Airbyte Growth Team on Vercel.

A member of the Team first needs to authorize it.

[Vignesh S (VigneshSankar01)]

Marcos Marx (@marcosmarxm) Christo Grabowski (@ChristoGrab) Natik Gadzhi (@natikgadzhi) for review

[Vignesh S (VigneshSankar01)]

relevant issue:
#55890

[Marcos Marx (marcosmarxm)]

I'm not sure if moving the fields properties will solve the issue Vignesh S (@VigneshSankar01) are you able to run a sync and it's working? Do you mind recording a video of working solution?

[Parthiv Makwana (parthiv11)]

Vignesh S (@VigneshSankar01) i think adding hardcoded fields is not preferred solution as it will make fields name static.

i tried and found that you cane just add {{ config['custom_reports_fields'].split(",") }} as fields value in json body

by default only id field is returned, just add fields you sync in custom_reports_fields which will make it dynamic as per usecase

[Marcos Marx (marcosmarxm)]

Closing as the proposal won't solve the issue.