-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Description
Which part? Which one?
Unknown
Description
I was working my way through the Frontend Error Monitoring Tutorial (https://docs.sentry.io/product/sentry-basics/integrate-frontend/) and have discovered that the code project forked and cloned from GitHub (https://github.com/getsentry/frontend-tutorial) is in an inconsistent state with the expectations laid out in the tutorial instructions.
For example, the step to capture a first error (https://docs.sentry.io/product/sentry-basics/integrate-frontend/generate-first-error/#1-trigger-an-error) instructs to open the src/components/App.js file and uncomment line 97 to enable the following line of code:
this.myCodeIsMorePerfect();
However, that line is already uncommented in the project code that has been pulled.
Another example is the section that goes over setting up readable stack traces (https://docs.sentry.io/product/sentry-basics/integrate-frontend/upload-source-maps/). The section about generating source maps (https://docs.sentry.io/product/sentry-basics/integrate-frontend/upload-source-maps/#1-generate-source-map-files-using-your-build-tool) instructs to add a block of code to webpack.config.js, however, the code is already there. It also mentions that line 11 can be uncommented. It already is. Line 11 (devtool: "source-map") also does not quite match with what is shown in the instructions (devtool: "hidden-source-map"). It is similar, but different. I don't know what effect that could have on behavior.
There are other inconsistencies as well, but the point I'm making is that the project in GitHub does not match the initial code state for modification by the student to match what is expected and described by the tutorial instructions. Unknown to me, there may be other differences in the code that change the behavior and expected outcome.
Suggested Solution
Please review the code projects in GitHub to ensure they are in a consistent state that matches the tutorial instructions. Please advise when these corrections have been made and pushed out.