Skip to content
This repository has been archived by the owner. It is now read-only.

Support for Sentry #415

Merged
merged 1 commit into from Jul 12, 2017
Merged

Support for Sentry #415

merged 1 commit into from Jul 12, 2017

Conversation

@RaitoBezarius
Copy link
Member

@RaitoBezarius RaitoBezarius commented Jul 9, 2017

Add Sentry's error reporting.

I'll add the necessary keys (on dissem.in and sandbox maybe) by myself once it's merged.

(if no key is specified, it won't interfere / throw an error / fail / crash / burn your house, no worries.)

Here the reference for Django
Theorically, keys should be added in secrets.py.

I should document this… 😄

Also, once I get approval from sentry.io for open source Sentry, I'll enable GitHub auth through the Dissemin org, so that everyone in the org can access to the Sentry Dissemin's org.

@RaitoBezarius RaitoBezarius requested a review from wetneb Jul 9, 2017
@wetneb
wetneb approved these changes Jul 9, 2017
Copy link
Member

@wetneb wetneb left a comment

That looks amazingly simple to set up!

@RaitoBezarius RaitoBezarius merged commit 44a5390 into master Jul 12, 2017
2 of 3 checks passed
2 of 3 checks passed
continuous-integration/travis-ci/pr The Travis CI build failed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
dependency-ci Dependencies checked
Details
@wetneb wetneb deleted the raito/sentry branch Feb 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants