Skip to content
This repository has been archived by the owner on May 16, 2024. It is now read-only.

Feat/custom axiom logging #25

Merged
merged 7 commits into from
Nov 15, 2023
Merged

Feat/custom axiom logging #25

merged 7 commits into from
Nov 15, 2023

Conversation

horvatz
Copy link
Member

@horvatz horvatz commented Nov 14, 2023

PR description

Add custom Axiom logger (client side + server side). Disabled console., use Axiom exclusively for logs.

Notion ticket links:

PR author checklist:

  • 🔫 PR is small and easy to review
  • 🧪 tests were added/updated where applicable
  • 📜 documentation was updated (README.md, Notion etc.)
  • 🔭 observability is sensible

Reviewer checklist:

  • 🧠 tested manually
  • 📱 tested on mobile (if applicable)
  • 🌐 tested on different browsers (if applicable)
  • 🔭 observability is sensible
  • 🤖 automatic tests are sufficient

After merge checklist:

  • 📡 client/project manager was notified about update

@horvatz horvatz requested a review from zigapk November 14, 2023 13:21
Copy link

vercel bot commented Nov 14, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
next-template ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 15, 2023 1:01pm

Copy link

github-actions bot commented Nov 14, 2023

Coverage report

St.
Category Percentage Covered / Total
🔴 Statements
56.95% (+2.41% 🔼)
86/151
🔴 Branches 40.54% 15/37
🔴 Functions
48% (+2.17% 🔼)
12/25
🔴 Lines
56.83% (+2.29% 🔼)
79/139
Show new covered files 🐣
St.
File Statements Branches Functions Lines
🟢
... / log-client.tsx
100% 100% 100% 100%

Test suite run success

8 tests passing in 2 suites.

Report generated by 🧪jest coverage report action from 441eba9

@zigapk zigapk merged commit 9ef4501 into master Nov 15, 2023
9 checks passed
@zigapk zigapk deleted the feat/custom-axiom-logging branch November 15, 2023 13:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants