Skip to content
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

documentation/streamlit-report-addition #149

Conversation

fivetran-joemarkiewicz
Copy link
Contributor

Addition of the Streamlit app to the README as it seems it was either erroneously removed or was never added.

@fivetran-joemarkiewicz fivetran-joemarkiewicz marked this pull request as ready for review April 15, 2024 15:18
Copy link
Contributor

@fivetran-reneeli fivetran-reneeli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@fivetran-joemarkiewicz fivetran-joemarkiewicz merged commit f8c97e3 into bugfix/duplicate_sla_event_ids_from_dst Apr 15, 2024
8 checks passed
@fivetran-joemarkiewicz fivetran-joemarkiewicz deleted the documentation/streamlit-report-addition branch April 15, 2024 16:02
fivetran-jamie added a commit that referenced this pull request Jun 25, 2024
* cast from timestamp to date

* reduce week generate from 208 to 52

* Update int_zendesk__reply_time_combined.sql

try different order of operations

* update schedule spine to explode out holiday periods

* forgot to add dbt timestamp

* addd timestamp cast and update test schema

* guess this didnt get staged

* new schema?

* remove the extra parenthesis that was just used for testing

* documentation/streamlit-report-addition (#149)

* documentation/streamlit-report-addition

* Update CHANGELOG.md

* Update README.md

* no changes to reply time combined

* versioning and changelog

* feature/first_reply_time_hours_performance_fix (#152)

* feature/first_reply_time_hours_performance_fix

* cleanup adjustments and documentation

* fix for databricks

* changelog

* Update models/sla_policy/reply_time/int_zendesk__reply_time_business_hours.sql

Co-authored-by: Joe Markiewicz <74217849+fivetran-joemarkiewicz@users.noreply.github.com>

* decision log

* more updates on changelog

* up version

* docs

* bugfix/remianing-sla-issues

* validation additions

* changelog addition

* Update CHANGELOG.md

Co-authored-by: Renee Li <91097070+fivetran-reneeli@users.noreply.github.com>

* Update CHANGELOG.md

Co-authored-by: Renee Li <91097070+fivetran-reneeli@users.noreply.github.com>

* moving tests and adding new ones

* changelog update and requirements limit

* Update CHANGELOG.md

Co-authored-by: Renee Li <91097070+fivetran-reneeli@users.noreply.github.com>

* dics

* customer metrics drafted

* bk

* switch schedule

---------

Co-authored-by: Renee Li <renee.li@fivetran.com>
Co-authored-by: Renee Li <91097070+fivetran-reneeli@users.noreply.github.com>
Co-authored-by: Joe Markiewicz <74217849+fivetran-joemarkiewicz@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants