-
Notifications
You must be signed in to change notification settings - Fork 17
Adicionando sentry no projeto #48
Adicionando sentry no projeto #48
Conversation
✔️ Deploy Preview for dadosabertosdefeira ready! 🔨 Explore the source changes: 996222b 🔍 Inspect the deploy log: https://app.netlify.com/sites/dadosabertosdefeira/deploys/6133fe367de29700080e8169 😎 Browse the preview: https://deploy-preview-48--dadosabertosdefeira.netlify.app |
Tem que ver pq os checks estão falhando @matheusrocha89 Bom trabalho aqui! |
@anapaulagomes são as configurações para o netlify, por isso te pedi acesso. Já aceitei e estou corrigindo, vai ficar muito bem 😄 |
@anapaulagomes desculpa te incomodar outra vez 😄 preciso adicionar a integração do sentry com o netlify mas para isso pede que a pessoa tenha permissão de Owner ou Manager no Sentry 😄 Você pode fazer essa integração ou me dar esse acesso? |
Não incomoda não :) feito! Me avisa se deu certo.
…On Fri, 3 Sep 2021, 23:43 Matheus Cruz Rocha, ***@***.***> wrote:
@anapaulagomes <https://github.com/anapaulagomes> desculpa te incomodar
outra vez 😄 preciso adicionar a integração do sentry com o netlify mas
para isso pede que a pessoa tenha permissão de Owner ou Manager no Sentry
😄 Você pode fazer essa integração ou me dar esse acesso?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#48 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAOP3LUBA5YFZFOXC2MBXJTUAE6PJANCNFSM5DLQRWTA>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
|
@anapaulagomes certeza? Continua igual |
Tinha feito isso no Netlify haha tava viajando. Vê agora @matheusrocha89! |
…pect commit ativado
Tentando separar erros do preview e de produção
@anapaulagomes Parece que agora foi, e configurei por ambientes também. Se alguém puder revisar e dar um Ok poderemos integrar. Obrigado pela ajuda nas permissões 👍 🙇 |
@@ -2,6 +2,13 @@ | |||
command = "npm run build" | |||
publish = "out" | |||
|
|||
[context.production.environment] | |||
SENTRY_ENVIRONMENT = "production" | |||
NEXT_PUBLIC_SENTRY_ENVIRONMENT = "production" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Precisamos das duas variaveis @matheusrocha89 ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Aparentemente sim, porque quando colocava só uma, quando o erro acontecia no lado do browser ele não enviava para o environment correto por conta da variável não existir do lado do client.
Quando coloquei essa variável exposta para o client funcionou como deveria e os erros foram enviados para o environment de produção e os de PR foram para os de deploy-preview.
Só precisa atualizar a branch aqui e está pronto pro merge @matheusrocha89. Valeu! |
@anapaulagomes Feito, vou fazer merge. |
Esse PR adiciona o sentry ao projeto para capturar os erros.
closes #24