-
Notifications
You must be signed in to change notification settings - Fork 30
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Feature] Incoprorate Historical Schedule Changes #158
Comments
@fivetran-jamie Hi Jamie, we are a Fivetran customer and looking to rely on this feature for critical business reporting. What's the current status of this, and are we able to get this prioritized? Let me know and thank you so much. |
Hi @dkang1023 we are actually committing to this as a feature update in our upcoming quarter (starting next week). I estimate we will have a working version ready by mid September. |
Hi @fivetran-joemarkiewicz - I wanted to check-in here and confirm we should expect a fix in the upcoming weeks in September? Let me know and thanks again. |
Hi @dkang1023 I will picking up this ticket. The updates are a bit complex, so the goal is to have this finished around the end of September! |
hi @fivetran-catfritz - I wanted to check in and see if you would be able to provide an estimated ETA on this. Thank you very much for all of the support! |
Hi @robabramowitz thank you for checking in! And good timing--I am currently working on this and hope to have a test branch available in the next couple days. |
Hi @dkang1023 and @robabramowitz I have created a test branch with the updates to bring in historical schedules. If you would like to install it and try it out, you can add the following code in place of the existing zendesk in your packages.yml. - git: https://github.com/fivetran/dbt_zendesk.git
revision: feature/historical-schedules
warn-unpinned: false Let me how this works for you! If things are working out, this will move on to release. |
@fivetran-catfritz Thank you - we are doing validation checks on our end. I see that it has already been pushed to release, so we will follow up with our findings. Thank you for the patience. |
This update has since been rolled out in the latest version of the package. As such, I will close out this issue. |
Is there an existing feature request for this?
Describe the Feature
Account for changes made to schedule start/end times by using the
audit_log
source table. Work has been started in #157 and fivetran/dbt_zendesk_source#50 but the logic in the schedule_spine model needs to be updated to include the new codeDescribe alternatives you've considered
No response
Are you interested in contributing this feature?
Anything else?
No response
The text was updated successfully, but these errors were encountered: