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

feat(sentry): add sentry logging to all firebase calls #1755

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

SelmaBergstrand
Copy link
Collaborator

Legger til Sentry i alle firestore-kall (blant annet).

Verdt å merke seg:

  • når en uventet feil skjer i forbindelse med et skjema kjører vi captureException til Sentry, men brukeren får feilmelding i skjema som vanlig (siden kræsjer ikke)
  • når en uventet feil skjer utenfor et skjema, kan en av to ting skje. Enten kræsjer vi siden (throw error) og redirecter bruker til bever, eller så skjer det ingenting. Jeg er usikker på når vi skal gjøre hver av disse! Noen feil er kanskje ikke store nok til å kræsje hele siden, men per i dag har vi ingen måte (som ikke er toast) å vise noe til bruker utenfor et skjema.
  • Når vi sender bruker til bever vil den uhåndterte feilen uansett kastes til Sentry, men da kjører vi også captureMessage for å legge til tilleggsinfo (siden stack trace i seg selv noen ganger er uforståelig)
  • jeg har prøvd å legge til sentry kun på "nederste nivå" (e.g. der firebase kallet faktisk skjer) for å unngå dobbelrapportering av samme feil. Dvs at vi feks ikke trenger sentrykall på en del fetcher, for det ligger en trycatch inne i selve fetchern.

Gjerne dobbelsjekk litt grundig at ting funker siden det er så mange filer som er endra. Dobbelsjekk gjerne at jeg har husket å returnere handleError(). PS: redirect kaster i seg selv en error 🤠 så den må alltid utenfor trycatch

Tenker vi må få på plass en egen oppgave for å vise bruker feil utenfor skjema

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