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

chore: remove warning when parseDateFields reaches depth limit #521

Merged
merged 1 commit into from
Feb 26, 2024

Conversation

tobice
Copy link
Contributor

@tobice tobice commented Feb 26, 2024

This is polluting logs as there are data structures in API responses that have arbitrary depth (user data, data sets).

Context:
https://apifier.slack.com/archives/CGZSN9DQC/p1708707905914559

This is polluting logs as there are data structures in API responses
that have arbitrary depth (user data, data sets).

Context:
https://apifier.slack.com/archives/CGZSN9DQC/p1708707905914559
@tobice tobice added the adhoc Ad-hoc unplanned task added during the sprint. label Feb 26, 2024
@tobice tobice requested a review from B4nan February 26, 2024 09:12
@tobice tobice self-assigned this Feb 26, 2024
@github-actions github-actions bot added this to the 84th sprint - Platform team milestone Feb 26, 2024
@github-actions github-actions bot added the t-platform Issues with this label are in the ownership of the platform team. label Feb 26, 2024
@B4nan B4nan merged commit 1245deb into master Feb 26, 2024
6 checks passed
@B4nan B4nan deleted the tobik/chore/remove-parse-date-fields-depth-warning branch February 26, 2024 09:19
@tobice
Copy link
Contributor Author

tobice commented Feb 26, 2024

Issue for a proper solution: #522

@drobnikj drobnikj added the validated Issues that are resolved and their solutions fulfill the acceptance criteria. label Feb 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
adhoc Ad-hoc unplanned task added during the sprint. t-platform Issues with this label are in the ownership of the platform team. validated Issues that are resolved and their solutions fulfill the acceptance criteria.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants