Skip to content

Commit

Permalink
docs: update debugging sentry step 3 (#28034)
Browse files Browse the repository at this point in the history
## **Description**

When using the command `METAMASK_DEBUG=true` in the `.metamaskrc` file
and building locally with `yarn dist` or `yarn dist:mv2` MetaMask cannot
be loaded on Chrome or Firefox.

By updating Step 3 from `development/README.md` and using
`METAMASK_DEBUG=1` the issue is no longer present and the MetaMask
extension is loading normally on Chrome and Firefox.

[![Open in GitHub
Codespaces](https://github.com/codespaces/badge.svg)](https://codespaces.new/MetaMask/metamask-extension/pull/28034?quickstart=1)

## **Related issues**

Fixes: Loading issues for MetaMask extension when using
`METAMASK_DEBUG=true` and building locally with `yarn dist` or `yarn
dist:mv2`

## **Manual testing steps**

1. Use `METAMASK_DEBUG=1` in the `.metamaskrc` file
2. Build the project locally with `yarn dist` or `yarn dist:mv2`
3. Open the MetaMask extension and see that it's loading with no issues
on Chrome or Firefox

## **Screenshots/Recordings**


### **Before**


![Chrome](https://github.com/user-attachments/assets/acbe4e32-dae8-4fb5-9570-f89a0a155f0d)

![Firefox](https://github.com/user-attachments/assets/8bbecfe7-84cc-476f-8c0d-44d7f9daeb12)

### **After**

![Screenshot 2024-10-23 at 10 57
49](https://github.com/user-attachments/assets/4b3a18c5-4023-4203-83c4-d8cbaa21bcfa)


## **Pre-merge author checklist**

- [ ] I've followed [MetaMask Contributor
Docs](https://github.com/MetaMask/contributor-docs) and [MetaMask
Extension Coding
Standards](https://github.com/MetaMask/metamask-extension/blob/develop/.github/guidelines/CODING_GUIDELINES.md).
- [ ] I've completed the PR template to the best of my ability
- [ ] I’ve included tests if applicable
- [ ] I’ve documented my code using [JSDoc](https://jsdoc.app/) format
if applicable
- [ ] I’ve applied the right labels on the PR (see [labeling
guidelines](https://github.com/MetaMask/metamask-extension/blob/develop/.github/guidelines/LABELING_GUIDELINES.md)).
Not required for external contributors.

## **Pre-merge reviewer checklist**

- [ ] I've manually tested the PR (e.g. pull and build branch, run the
app, test code being changed).
- [ ] I confirm that this PR addresses all acceptance criteria described
in the ticket it closes and includes the necessary testing evidence such
as recordings and or screenshots.
  • Loading branch information
Unik0rnMaggie authored Oct 24, 2024
1 parent 2ae9236 commit 1698011
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion development/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ or `https://api.segment.io/v1/batch` respectively.

2. To display Sentry logs, include `DEBUG=metamask:sentry:*` in `.metamaskrc`.

3. To display more verbose logs if not in a developer build, include `METAMASK_DEBUG=true` in `.metamaskrc`.
3. To display more verbose logs if not in a developer build, include `METAMASK_DEBUG=1` in `.metamaskrc`.

4. Ensure metrics are enabled during onboarding or via `Settings > Security & privacy > Participate in MetaMetrics`.

Expand Down

0 comments on commit 1698011

Please sign in to comment.