Add Optional utc_offset Parameter to Adjust Source

bruin-data / ingestr

ingestr is a CLI tool to copy data between any databases with a single command seamlessly.

https://bruin-data.github.io/ingestr/

MIT License

2.57k stars 62 forks source link

Add Optional utc_offset Parameter to Adjust Source #51

Closed saidkaban closed 1 week ago

saidkaban commented 1 week ago

This PR adds an optional utc_offset parameter to the Adjust source, allowing users to specify a UTC offset for data retrieval. If not provided, it defaults to "+00:00".

Key Changes:

URI Parsing: Added utc_offset parsing in AdjustSource.
Source Function: Updated adjust_source to accept utc_offset.
Helper Function: Set default utc_offset to "+00:00" in fetch_report_data.
Documentation: Updated Adjust source docs to include utc_offset.

karakanb commented 1 week ago

I just merged https://github.com/bruin-data/ingestr/pull/50 which covers this usecase as well. thanks a lot for the PR!