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.

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

Integrate sentry for Simulator #4962

Open
tachyons opened this issue May 30, 2024 · 14 comments · May be fixed by CircuitVerse/cv-frontend-vue#391
Open

Integrate sentry for Simulator #4962

tachyons opened this issue May 30, 2024 · 14 comments · May be fixed by CircuitVerse/cv-frontend-vue#391
Labels

Comments

@tachyons
Copy link
Member

Push errors to sentry for better tracking.

@tachyons tachyons added the good first issue Good for newcomers label Oct 15, 2024
@tachyons
Copy link
Member Author

@rzeta-10
Copy link

/assign

@rzeta-10
Copy link

@tachyons sir could you please review the merge request

@mrsingh-rishi
Copy link

/assign

1 similar comment
@akshayw1
Copy link

/assign

@akshayw1
Copy link

Really intrested to work on the issue, Can I?

@salmoneatenbybear
Copy link
Contributor

is there an existing Sentry DSN ? @tachyons

@Akshat1000Sharma
Copy link

Since this issue is open, plz assign this to me

AdityaP700 added a commit to AdityaP700/CircuitVerse that referenced this issue Nov 25, 2024
- Add Sentry configuration with environment-specific settings
- Configure error tracking and performance monitoring
- Add proper canvas initialization
- Enable component tracking and error sampling

Fixes CircuitVerse#4962
AdityaP700 added a commit to AdityaP700/CircuitVerse that referenced this issue Nov 25, 2024
Implement error monitoring and performance tracking for the simulator:
- Add Sentry configuration with environment-specific settings
- Fix canvas initialization and error handling
- Configure error sampling and component tracking
- Enable performance monitoring for Vue components

Closes CircuitVerse#4962
@TejasNangru
Copy link

There’s already a pull request but I would like to know if additional contributions are needed.
@tachyons

@H-ario-m
Copy link

H-ario-m commented Jan 3, 2025

Since there is already a open pull request I would like to ask if there is any additional contributions , documentation or enhancement needed @tachyons

@Recxsmacx
Copy link

./assign

@Recxsmacx
Copy link

/assign

Recxsmacx added a commit to Recxsmacx/CircuitVerse that referenced this issue Feb 1, 2025
Related to CircuitVerse#4962

Integrate Sentry error handling to capture and report errors in the application.

* **ApplicationController**:
  - Add Sentry error handling to capture and report errors.
  - Update `auth_error` method to send errors to Sentry.
  - Update `custom_auth_error` method to send errors to Sentry.

* **Api::V1::SimulatorController**:
  - Add Sentry error handling to capture and report errors.
  - Update `post_issue` method to send errors to Sentry.
  - Update `verilog_cv` method to send errors to Sentry.

* **ErrorsController**:
  - Add Sentry error handling to capture and report errors.
  - Update `not_found` method to send errors to Sentry.
  - Update `unacceptable` method to send errors to Sentry.
  - Update `internal_error` method to send errors to Sentry.

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/CircuitVerse/CircuitVerse/issues/4962?shareId=XXXX-XXXX-XXXX-XXXX).
@ThatDeparted2061
Copy link
Member

ThatDeparted2061 commented Mar 1, 2025

@salmoneatenbybear What are the updates here for the issue ?

@salmoneatenbybear
Copy link
Contributor

Another contributor superseded this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment