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

sentry.repl.it #258

Closed
wants to merge 1 commit into from
Closed

sentry.repl.it #258

wants to merge 1 commit into from

Conversation

fabianski7
Copy link

58

@lightswitch05
Copy link
Owner

I will not be accepting this PR. Please see #58 - especially #58 (comment)

@lightswitch05 lightswitch05 added broken-site Valid block breaks a legit site wontfix This will not be worked on labels Feb 9, 2021
@fabianski7
Copy link
Author

Yeah, I read it.
You said you will keep blocking the domain until they make some changes. But I didn't understand the reason to reject this domain and have several others already included.

@lightswitch05
Copy link
Owner

This is not a required domain to operate the website. I suggest trying to reach someone at repl.it to add proper error handling around the Sentry code so that the website does not break if Sentry fails to load. Looking at their Github perhap @kochman would be willing to forward this bug to someone who can fix it

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
broken-site Valid block breaks a legit site wontfix This will not be worked on
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@fabianski7 @lightswitch05