airbytehq / airbyte

The leading data integration platform for ETL / ELT data pipelines from APIs, databases & files to data warehouses, data lakes & data lakehouses. Both self-hosted and Cloud-hosted.
https://airbyte.com
Other
16.07k stars 4.11k forks source link

[source-smartsheets]: Add support for reading from Reports #35197

Open bffrost opened 8 months ago

bffrost commented 8 months ago

Connector Name

source-smartsheets

Connector Version

1.1.2

What step the error happened?

Configuring a new connector

Relevant information

Background

I'm using the Smartsheet Source and have some data that is heavily aggregated from many source Sheets into a Report. I'd like to be able to read from the Report.

Steps to Reproduce:

Expected Results:

Actual Results

Extra Information

I am preparing a PR to add this enhancement. I'm aware that the Smartsheet API doesn't allow querying Report rows by modified date, so I've implemented only a Full Refresh sync mode.

Relevant log output

No response

Contribute

bffrost commented 8 months ago

@davydov-d Are you open to a community PR from me for adding this feature?

marcosmarxm commented 8 months ago

@bffrost do you want to contribute to solve this issue?

bffrost commented 8 months ago

@marcosmarxm yes, I do. I have a fix prepared in a fork of the airbyte repo but haven't created a PR yet since the contribution guidelines mentioned PRs not linked to triaged issues would be rejected.

octavia-squidington-iii commented 1 month ago

At Airbyte, we seek to be clear about the project priorities and roadmap. This issue has not had any activity for 180 days, suggesting that it's not as critical as others. It's possible it has already been fixed. It is being marked as stale and will be closed in 20 days if there is no activity. To keep it open, please comment to let us know why it is important to you and if it is still reproducible on recent versions of Airbyte.

bffrost commented 1 month ago

Sorry for the delay, but I've finally submitted a PR with my fix for this