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

Configure Sentry to support error reporting with Source Maps #157

Closed
vuhuucuong opened this issue Jan 31, 2020 · 1 comment · Fixed by #188
Closed

Configure Sentry to support error reporting with Source Maps #157

vuhuucuong opened this issue Jan 31, 2020 · 1 comment · Fixed by #188
Assignees
@duong-se
Labels
aml-checklist Relates to Anti-money laundering app enhancements Relates to a feature enhancement front-end Relates to front end issues geo-diary Relates to GEO Diary app lifetime-legal Relates to Lifetime Legal App marketplace Relates to the Marketplace
Milestone
Foundations Alpha...

Comments

@vuhuucuong
Copy link
Contributor

vuhuucuong commented Jan 31, 2020 •
edited
Loading

Summary: Currently, the reported errors from Sentry were unreadable since it's reporting errors from the compiled source code.
image

Task: Configure Sentry to support Source Maps, so it'll report correctly where the error comes from in our actual source code before it was compiled

Reference link: https://docs.sentry.io/platforms/javascript/sourcemaps/
@vuhuucuong vuhuucuong added marketplace Relates to the Marketplace lifetime-legal Relates to Lifetime Legal App geo-diary Relates to GEO Diary app aml-checklist Relates to Anti-money laundering app enhancements Relates to a feature enhancement labels Jan 31, 2020
@duong-se duong-se self-assigned this Feb 4, 2020
@duong-se duong-se mentioned this issue Feb 6, 2020
Merged
@duong-se
Copy link
Contributor

duong-se commented Feb 6, 2020

Hi @willmcvay could you help me to add token to secret of github
you can get token via this doc https://docs.sentry.io/cli/configuration/

SENTRY_AUTH_TOKEN=

@HollyJoyPhillips HollyJoyPhillips added the front-end Relates to front end issues label Feb 10, 2020
duong-se added a commit that referenced this issue Feb 11, 2020
@duong-se
feat: #157 config for sentry log support source mapp
91fa67a
duong-se added a commit that referenced this issue Feb 11, 2020
@duong-se
feat: #157 config for sentry log support source mapp (#188)
d1405fa
nphivu414 pushed a commit that referenced this issue Apr 29, 2020
Merge pull request #157 from reapit/feature/CLD-431-sync-develop
7b5a6f2
nphivu414 pushed a commit that referenced this issue Apr 29, 2020
@duong-se
feat: #157 config for sentry log support source mapp (#188)
faa74fe
@willmcvay willmcvay mentioned this issue May 13, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@duong-se duong-se

Labels
aml-checklist Relates to Anti-money laundering app enhancements Relates to a feature enhancement front-end Relates to front end issues geo-diary Relates to GEO Diary app lifetime-legal Relates to Lifetime Legal App marketplace Relates to the Marketplace
Projects
None yet
Milestone
Foundations Alpha Developer Release
Development

Successfully merging a pull request may close this issue.

feat: #157 config for sentry log support source map reapit/foundations
3 participants
@duong-se @vuhuucuong @HollyJoyPhillips