search

NIAEFEUP / nijobs-fe

[FRONTEND] A platform for companies to advertise their job opportunities to students
GNU General Public License v3.0
23 stars 3 forks source link

Added initial config with sentry to report errors #228

Closed monkin77 closed 2 years ago

monkin77 commented 2 years ago

closes #226

monkin77 commented 2 years ago

What is the added value of Sentry, since we are already posting metrics to google analytics, including the errors?

The main reason was to be notified when the server went down

imnotteixeira commented 2 years ago

I'd go with this instead. https://niaefeup.slack.com/apps/A9Z9ZNFE1-uptimebot?tab=more_info

Adding sentry seems a bit overkill, and it adds up to the dependencies/bundle size, which makes the pages load slower

monkin77 commented 2 years ago

I'd go with this instead. https://niaefeup.slack.com/apps/A9Z9ZNFE1-uptimebot?tab=more_info

Adding sentry seems a bit overkill, and it adds up to the dependencies/bundle size, which makes the pages load slower

sounds good