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 new expected Error INTEGRATION_ACCOUNT_EXPIRED_ERROR. No Logs i… #94

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

siavashcsr
Copy link
Collaborator

…f CRM Account is expired

@siavashcsr siavashcsr requested a review from LemonaOna May 9, 2023 13:54
@LemonaOna
Copy link
Contributor

hab das mal refactored damit man das nicht für jeden Error einzeln machen muss. Aber dann hab ich mich gefragt, ob man das wirklich für alle diese Errors machen will (nicht loggen) oder ob das nur für den Sonderfall IntegrationRefreshError galt (der ist ja 400 und nicht 452)

@siavashcsr
Copy link
Collaborator Author

hab das mal refactored damit man das nicht für jeden Error einzeln machen muss. Aber dann hab ich mich gefragt, ob man das wirklich für alle diese Errors machen will (nicht loggen) oder ob das nur für den Sonderfall IntegrationRefreshError galt (der ist ja 400 und nicht 452)

Tja gute Frage, vielleicht wollen wir uns erst mal rantasten und die Fehler einzeln aus dem Logging entfernen? Da fehlt uns noch etwas die Erfahrung würde ich denken. Mir sind im Augenblick noch zu viel Logs (und damit auch zuviel Alarme) lieber als zu wenig.

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