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

Added fixes while testing against CRM build environment data #1751

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

hortha
Copy link
Contributor

@hortha hortha commented Dec 16, 2024

Context

As the SyncAllInductionsFromCrmJob was run against more volumes of data, various issues came to light which needed fixing.

Changes proposed in this pull request

Various tweaks to ensure that all induction data and related audit history for dfeta_induction and contact table get synced correctly.

Checklist

  • Attach to Trello card
  • Rebased master
  • Cleaned commit history
  • Tested by running locally
  • Run DQT integration tests locally (if appropriate)

@hortha hortha requested a review from gunndabad December 16, 2024 14:14
@hortha
Copy link
Contributor Author

hortha commented Dec 16, 2024

Removed temporary code to avoid running the sync job against all data in CRM build

Skip test which is failing in CI for the moment

Added logging to sync helper to highlight when we're getting CRM rate limiting

Added more logging when rate limiting is hit
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