Skip to content

chore: update logging for the CSV export function#1211

Merged
davidgamez merged 1 commit intomainfrom
chore/improve_logs_export_csv
May 26, 2025
Merged

chore: update logging for the CSV export function#1211
davidgamez merged 1 commit intomainfrom
chore/improve_logs_export_csv

Conversation

@davidgamez
Copy link
Copy Markdown
Member

@davidgamez davidgamez commented May 23, 2025

Summary:

Part of #1027
This PR updates the logging calls to the new standardized pattern on the export csv function.

Expected behavior:

The Logger class is no longer used in favour of the newly added logging functions.

Testing tips:

This can only be visible in the life environment.

  • Deploy to DEV environment
  • Tigger the functions
  • Verify that the logs are properly formatted with trace and span IDs.

Please make sure these boxes are checked before submitting your pull request - thanks!

  • Run the unit tests with ./scripts/api-tests.sh to make sure you didn't break anything
  • Add or update any needed documentation to the repo
  • Format the title like "feat: [new feature short description]". Title must follow the Conventional Commit Specification(https://www.conventionalcommits.org/en/v1.0.0/).
  • Linked all relevant issues
  • Include screenshot(s) showing how this pull request works and fixes the issue(s)

@davidgamez davidgamez changed the title chore: update logging chore: update logging for the CSV export function May 23, 2025
@davidgamez
Copy link
Copy Markdown
Member Author

Log records:
Screenshot 2025-05-23 at 12 40 51 PM

@davidgamez davidgamez requested a review from jcpitre May 23, 2025 16:42
@davidgamez davidgamez merged commit 6b7eb9b into main May 26, 2025
9 checks passed
@davidgamez davidgamez deleted the chore/improve_logs_export_csv branch May 26, 2025 13:24
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