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

TDL-19802: Add new streams talk_phone_numbers and ticket_metrics_event #111

Conversation

jtilala
Copy link
Contributor

@jtilala jtilala commented Jul 6, 2022

Description of change

Manual QA steps

  • Test Discovery mode and check-in catalog that we are getting proper schema.
  • Run sync mode and check whether we are getting data in the proper format for the newly added stream or not.

Risks

Rollback steps

  • revert this branch

@kethan1122 kethan1122 merged commit 7153890 into master Jun 14, 2023
dsprayberry added a commit that referenced this pull request Jun 14, 2023
zachharris1 pushed a commit that referenced this pull request Jun 14, 2023
#111)

* TDL-19802: Add new streams talk_phone_numbers and ticket_metrics_event

* resolevd pylint error

* add unittest

* add unitests for both newly added streams

* add streams in base.py

* remove exception

* exclude ola field from test_all_field test case

* remove token field from talk_pjone_numbers all field test case

* updated start_date in base.py

* ignore talk_phone_numbers for bookmark test

* add new fields

* change start date

* change start date

* add backoff for connectionReseterror and chunkencoding error

* add backoff for connectionReseterror and chunkencoding error

* add backoff for connectionReseterror and Protocol error

* connection reset error

* remove backoffs

* revert http changes and all fields test

* revert http changes

* revert http changes

* use builtin libraries

* merge with master

* adjust integration tests

* adjust pagination test

* make time as date-time format

* chnagelog and version bump

---------

Co-authored-by: Jay Tilala <[email protected]>
Co-authored-by: kethan1122 <[email protected]>
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.

3 participants