Skip to content
This repository has been archived by the owner on Mar 13, 2023. It is now read-only.

feat: Sentry Integration #602

Merged
merged 5 commits into from
Aug 22, 2022
Merged

feat: Sentry Integration #602

merged 5 commits into from
Aug 22, 2022

Conversation

silasary
Copy link
Collaborator

@silasary silasary commented Aug 14, 2022

What type of pull request is this?

  • Non-breaking code change
  • Breaking code change
  • Documentation change/addition
  • Tests change

Description

Adds an extension that sets up Sentry.io hooks
Finally Closes #403

Changes

Adds extension.
Adds (really bad) documentation. Polls, please rewrite improve it.

Checklist

  • I've formatted my code with Black
  • I've ensured my code works on Python 3.10.x
  • I've tested my code

@silasary silasary added Improvement Something that is already implemented but can be done better New Feature::Library A new feature for the library and removed Improvement Something that is already implemented but can be done better labels Aug 14, 2022
naff/ext/sentry.py Outdated Show resolved Hide resolved
docs/src/Guides/24 Error Tracking.md Outdated Show resolved Hide resolved
naff/ext/sentry.py Outdated Show resolved Hide resolved
@LordOfPolls
Copy link
Member

Docs could be improved, but i'm happy to merge (changes pending) without their improvement as theyre succinct enough

@silasary silasary merged commit 1ec6afc into NAFTeam:dev Aug 22, 2022
@silasary silasary deleted the sentry branch August 22, 2022 06:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
New Feature::Library A new feature for the library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants