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

Added Sentry Monitoring #176

Closed
wants to merge 2 commits into from
Closed

Added Sentry Monitoring #176

wants to merge 2 commits into from

Conversation

tsu-ki
Copy link
Contributor

@tsu-ki tsu-ki commented Dec 14, 2024

Resolves #(#112)

@arkid15r can you please specify if I should create a DNS for Sentry, or one already exists for it.
I'll test the proposed changes first and then update the PR accordingly

@yashpandey06
Copy link
Contributor

@tsu-ki run pre-commit

@yashpandey06
Copy link
Contributor

yashpandey06 commented Dec 16, 2024

@tsu-ki any updates regarding the changes ...?

cc : @arkid15r

@tsu-ki
Copy link
Contributor Author

tsu-ki commented Dec 16, 2024

@yashpandey06 I’ll push the changes tomorrow night as I have an interview scheduled for tomorrow afternoon and I’ve been busy preparing for that. My apologies for the delay

@yashpandey06
Copy link
Contributor

@yashpandey06 I’ll push the changes tomorrow night as I have an interview scheduled for tomorrow afternoon and I’ve been busy preparing for that. My apologies for the delay

Thanks for the updates ... actually we are sort of blocked on an issue for which we need sentry ...any way we can wait ...best of luck !!

@tsu-ki
Copy link
Contributor Author

tsu-ki commented Dec 16, 2024

@yashpandey06 I apologise for that. If it’s an urgent task that needs to be resolved then please ping me on slack and I’ll coordinate the changes I’ve made so far and you can continue to work from there

@arkid15r
Copy link
Collaborator

This has been addressed in #187

@arkid15r arkid15r closed this Dec 19, 2024
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.

3 participants