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

Integrate Sentry.io error reporting #237

Open
francisli opened this issue Aug 29, 2024 · 2 comments · May be fixed by #245
Open

Integrate Sentry.io error reporting #237

francisli opened this issue Aug 29, 2024 · 2 comments · May be fixed by #245
Assignees

Comments

@francisli
Copy link
Collaborator

Post in #intentional-walk-dev-chatter with an email address to be invited to our project in Sentry.io.

Then, integrate the Python/Django error reporting code as per their getting started guide.

https://sfcivictech.sentry.io/projects/intentional-walk-server/getting-started/?product=performance-monitoring&product=profiling

@sfbrigade sfbrigade deleted a comment Aug 29, 2024
@JimmiHagen JimmiHagen self-assigned this Sep 9, 2024
@JimmiHagen
Copy link
Collaborator

We will also want to instrument the app itself - this gives insight into on-device errors and may be more useful in this project than server-side logs : https://docs.sentry.io/platforms/react-native/#install
but that's a separate issue

@JimmiHagen
Copy link
Collaborator

changes are pushed to branch
tests run, only issue is known, related to that other branch I should have fixed up by now
=========================================================== short test summary info ============================================================
FAILED home/tests/unit/api/test_appuser.py::TestIsTester::test_is_tester - AssertionError: True != False : failed 'iwt-d'
================================================== 1 failed, 79 passed, 17 warnings in 23.97s ==================================================

@JimmiHagen JimmiHagen linked a pull request Sep 12, 2024 that will close this issue
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 a pull request may close this issue.

2 participants