Language-Mapping / language-map

Front-end codebase for Language Mapping web map
https://languagemap.nyc
MIT License
6 stars 4 forks source link

Do more with Sentry #90

Open abettermap opened 4 years ago

abettermap commented 4 years ago

Do more with Sentry

Just about everything should be in here along with some other likely goodies as well.

Related

48 was a good start:

abettermap commented 4 years ago

Source maps seem to be working now that Netlify env is set. Can't really test without errors though...

image

abettermap commented 4 years ago

@rperlin-ela Sentry is working great (although part of me misses the "ignorance is bliss" days before I knew about the errors 😄 ) but I'm struggling with how to prioritize some of the errors when I don't know what percentage of users are experiencing them. I know how many users are having the errors, but is iiiiiiit 6 users out of 6 for a certain error, or 6 users out of... 60?

If it's like one dude having a ton of problems, I think the "oh well" approach is warranted. If it's tons of users, then the "oh? well. guess i need to fix that" is logical.

Anyway, Sentry has Performance monitoring capabilities, but not on our sponsored plan. I'm not sure that's what we want anyway, so perhaps this deserves to be broken out into a Google Analytics issue (small reference in #78). Then I could say "ok, 40 people hit the page that day, but only 2 had a particular error".

It's not terribly hard to set up GA, at least for a baseline thing like this. And it's excellent info (albeit creepy and stalky, but by that token most websites on the internet fall into that category) that would help me prioritize bug handling.

If you're on board I can add it to the SOW and create an issue and give you a link on how to set it up in Google (there is also Google Tag Manager that may be part of the process).

Thoughts?

rperlin-ela commented 4 years ago

New territory for me but sounds very reasonable and I’m happy to do whatever’s needed on this end. Only question is whether that’s something for this SOW or the next one, which is where I’m thinking all the GA stuff as well would go.

On Oct 12, 2020, at 10:51 PM, Jason Lampel notifications@github.com wrote:

@rperlin-ela https://github.com/rperlin-ela Sentry is working great (although part of me misses the "ignorance is bliss" days before I knew about the errors 😄 ) but I'm struggling with how to prioritize some of the errors when I don't know what percentage of users are experiencing them. I know how many users are having the errors, but is iiiiiiit 6 users out of 6 for a certain error, or 6 users out of... 60?

If it's like one dude having a ton of problems, I think the "oh well" approach is warranted. If it's tons of users, then the "oh? well. guess i need to fix that" is logical.

Anyway, Sentry has Performance monitoring capabilities, but not on our sponsored plan. I'm not sure that's what we want anyway, so perhaps this deserves to be broken out into a Google Analytics issue (small reference in #78 https://github.com/Language-Mapping/language-map/issues/78). Then I could say "ok, 40 people hit the page that day, but only 2 had a particular error".

It's not terribly hard to set up GA, at least for a baseline thing like this. And it's excellent info (albeit creepy and stalky, but by that token most websites on the internet fall into that category) that would help me prioritize bug handling.

If you're on board I can add it to the SOW and create an issue and give you a link on how to set it up in Google (there is also Google Tag Manager that may be part of the process).

Thoughts?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/Language-Mapping/language-map/issues/90#issuecomment-707451233, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMNKB5AKGYJTKGKKV6XXTTLSKO6CVANCNFSM4RXVQLFA.

abettermap commented 4 years ago

Sounds good. I'd really appreciate knowing how many users are visiting so that I can prioritize Sentry error fixes (there have been around half a dozen unique ones I think), so when you have a chance please follow the steps in the issue I assigned to you and let me know what the GTM tag ID and GA tracking ID are: #108.

When you've knocked that out, I'll re-assign the issue to myself and then implement it in the code.

abettermap commented 4 years ago

@rperlin-ela I created a custom alert in Sentry for the scenario of video/audio not being found via API, e.g. due to typo or maybe privacy settings or some of the scenarios in #72 and #116:

image

It's not bulletproof but it should send emails whenever that occurs. I get emails already but I added myself just as a check. If you'd prefer a different email address let me know and I can create a separate acct.

The other custom scenarios (and not all of these trigger a true error, I just added some as a convenience so we can catch data errors):

  1. URL is not one of the three expected formats: youtube video/playlist embed, internet archive embed
  2. some other failure