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

chore: sentry logging all endpoints #793

Merged
merged 1 commit into from
Jun 6, 2024

Conversation

dalsontws
Copy link
Contributor

@dalsontws dalsontws commented Jun 6, 2024

Notion link

This PR adds...

  • sentry error logging on all endpoints
  • I've kept this PR as small as possible (~600 lines) by splitting it into PRs with manageable chunks of code
  • I've requested reviews from 1-2 reviewers
  • I've added accessibility IDs to my components
  • I've added/updated unit/integration tests
  • I've added/updated e2e tests for at least the happy flow
  • I've added Chinese translations for i18n setup
  • I've added documentation in README/Notion

@supplyally-bot
Copy link

Application

Expo QR
OR Enter this link into your dev client: exp+rationally://expo-development-client/?url=https://u.expo.dev/cf511b6b-f66b-45b5-b856-2339182f8052?channel-name=pr793

Storybook

Expo QR
OR Enter this link into your dev client: exp+rationally://expo-development-client/?url=https://u.expo.dev/cf511b6b-f66b-45b5-b856-2339182f8052?channel-name=storybook-pr793

@dalsontws dalsontws force-pushed the chore/sentry-logging-all-endpoints branch from eaab20f to 983fbd5 Compare June 6, 2024 08:26
@dalsontws dalsontws self-assigned this Jun 6, 2024
@dalsontws dalsontws merged commit 8fd4f21 into master Jun 6, 2024
8 checks passed
@ilhaoni ilhaoni deleted the chore/sentry-logging-all-endpoints branch August 6, 2024 04:01
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.

3 participants