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.

Sign up for GitHub

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

Jump to bottom

Добавить коллектор ошибок Sentry #529

Open
fey opened this issue Oct 31, 2022 · 6 comments
Open

Добавить коллектор ошибок Sentry #529

fey opened this issue Oct 31, 2022 · 6 comments
Assignees
@dzencot

Comments

@fey
Copy link
Collaborator

fey commented Oct 31, 2022

https://docs.sentry.io/product/integrations/deployment/vercel/
https://docs.sentry.io/platforms/javascript/guides/nextjs/

Добавить отлов ошибок.
@jprestor
Copy link
Contributor

jprestor commented Nov 1, 2022

Сделаю её

@jprestor
Copy link
Contributor

jprestor commented Nov 2, 2022 •
edited

Понял что поторопился взяв эту задачу. Интеграцию с vercel лучше делать тому у кого есть доступы к аккаунтам vercel и sentry. Инициализацию в самом проекте сделал, все работает, но есть ли смысл делать PR? Там буквально две cli команды выполнить и добавить опции в next.config.js.

@fey
Copy link
Collaborator Author

fey commented Nov 2, 2022

Если должен появиться какой-то код - присылайте ПР.
Если нужно делать какую-то интерграцию на vercel, чтобы работала отправка ошибок в Sentry - опишите это. Это ведь не принципиально, Есть ли доступ или нет) Просто свое приложение поднимаете и на нем показываете пример =)

jprestor added a commit to jprestor/hexletguides.github.io that referenced this issue Nov 3, 2022
@jprestor
[Hexlet#529] init sentry
bdecdfc
jprestor added a commit to jprestor/hexletguides.github.io that referenced this issue Nov 3, 2022
@jprestor
[Hexlet#529] remove duplicates of .sentryclirc
cc60f4d
jprestor added a commit to jprestor/hexletguides.github.io that referenced this issue Nov 3, 2022
@jprestor
[Hexlet#529] comment sentry config
c98f66b
ashikov added a commit that referenced this issue Nov 25, 2022
@ashikov
Merge pull request #530 from jprestor/add-sentry-error-tracking
aa7c633 
[#529] Add sentry error tracking
@acidmaksim
Copy link
Contributor

@dzencot закрываю issues?

@acidmaksim
Copy link
Contributor

@dzencot удалось проверить?

@dzencot
Copy link
Collaborator

dzencot commented May 11, 2023

Пока не работает. Настроил на свой центри, но ошибки не приходят. Надо разбираться.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dzencot dzencot

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@fey @dzencot @jprestor @acidmaksim