twenty icon indicating copy to clipboard operation
twenty copied to clipboard

feat: add user to sentry

Open magrinj opened this issue 1 year ago • 1 comments

🚧 WIP 🚧

Fix #3468

magrinj avatar Jan 15 '24 16:01 magrinj

Cool! Does that also solve performance tracing (and eventually profiling which seems to be enabled too) ? I see it's supposed to be enabled in the code but not working in the Sentry console. I think this would be nice to prioritize it this week (we have performance on the frontend but I feel it's harder to interpret than if we had the backend side)

FelixMalfait avatar Jan 28 '24 10:01 FelixMalfait

@FelixMalfait Yes we can add tracing in another PR, as I've managed to find a solution here based on useSentry hook of graphql-yoga !

magrinj avatar Jan 31 '24 16:01 magrinj

TODOs/FIXMEs:

  • ~~// TODO: inject our own logger service to handle the output (Sentry, etc.)~~: packages/twenty-server/src/command.ts

Generated by :no_entry_sign: dangerJS against f5484686ebef70b07c98e1be5cc0161691dcc764

github-actions[bot] avatar Feb 01 '24 09:02 github-actions[bot]