-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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.
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
docs: Include Sentry Webpack Plugin in the Source Maps section #1155
Conversation
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.
Basically "you you", but maybe you can improve on the other two points.
docs/sourcemaps.rst
Outdated
|
||
Another way to manage releases and sourcemaps on Sentry, is to use the Sentry Webpack Plugin. | ||
|
||
* Start by creating a new authentication token under **[Account] > API**. |
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.
You could add a link to https://sentry.io/api/
here
docs/sourcemaps.rst
Outdated
Another way to manage releases and sourcemaps on Sentry, is to use the Sentry Webpack Plugin. | ||
|
||
* Start by creating a new authentication token under **[Account] > API**. | ||
* Ensure you you have ``project:write`` selected under scopes. |
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.
you you
docs/sourcemaps.rst
Outdated
* Start by creating a new authentication token under **[Account] > API**. | ||
* Ensure you you have ``project:write`` selected under scopes. | ||
* Install ``@sentry/webpack-plugin`` using ``npm`` | ||
* Create ``.sentryclirc`` file with necessary config (see Sentry Webpack Plugin docs below) |
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.
This can be confusing. @HazAT can sentry-wizard help here?
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.
So yeah, sentry-wizard
could help here, it would create the file for you but it is not yet implemented for "generic" projects.
If install @sentry/wizard
and select "Create Sentry Cli config" I could make it so it creates the file locally.
Also, I am not sure if we should promote sentry.properties
or .sentryclirc
or both?
@mitsuhiko @jan-auer
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.
I want to change sentry-cli first to always load sentry.properties
before we promote that more.
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.
Thanks. Merging it for now and we'll update as we go.
0f8826a
to
24d24fc
Compare
Related: getsentry/raven-node#400