Skip to content

fix: incorrect created at timestamp in a request#5

Merged
krzysztoff1 merged 2 commits intomainfrom
fix-incorrect-create-at-formatting
May 22, 2025
Merged

fix: incorrect created at timestamp in a request#5
krzysztoff1 merged 2 commits intomainfrom
fix-incorrect-create-at-formatting

Conversation

@krzysztoff1
Copy link
Copy Markdown
Collaborator

@krzysztoff1 krzysztoff1 commented May 22, 2025

  • refactored logger so that timestamps in console will always match those in dashboard
  • fix incorrect createdAt at param when running logger in non rails applications
  • add specs for http log sync
  • add coverage report after running specs (now coverage it at 100% [bundle exec rspec]) - later we should add more robust error handling
  • bump package to 1.0.2
  • add required dependency version to prevent gem warnings

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 22, 2025

⚠️ No Changeset found

Latest commit: 42b835e

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@krzysztoff1 krzysztoff1 self-assigned this May 22, 2025
@krzysztoff1 krzysztoff1 changed the title Fix incorrect created at timestamp in a request fix: incorrect created at timestamp in a request May 22, 2025
@krzysztoff1 krzysztoff1 marked this pull request as ready for review May 22, 2025 18:18
@krzysztoff1 krzysztoff1 force-pushed the fix-incorrect-create-at-formatting branch from f366bc5 to 8c17278 Compare May 22, 2025 18:25
@krzysztoff1 krzysztoff1 force-pushed the fix-incorrect-create-at-formatting branch from 8c17278 to 0e390fd Compare May 22, 2025 18:25
@krzysztoff1 krzysztoff1 merged commit c7e5462 into main May 22, 2025
@krzysztoff1 krzysztoff1 deleted the fix-incorrect-create-at-formatting branch May 22, 2025 19:25
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