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

Plugin for Pinia 🍍 #13279

Closed
smeubank opened this issue Aug 8, 2024 · 1 comment · Fixed by #13841
Closed

Plugin for Pinia 🍍 #13279

smeubank opened this issue Aug 8, 2024 · 1 comment · Fixed by #13841
Assignees
@onurtemizkan
Labels
Package: nuxt Issues related to the Sentry Nuxt SDK Package: vue Issues related to the Sentry Vue SDK

Comments

@smeubank
Copy link
Member

smeubank commented Aug 8, 2024 •
edited
Loading

Problem Statement

Pinia is the official solution for more advanced or larger scale state management in Vue applications.

Pinia can also used in Nuxt SSR

The Sentry

Solution Brainstorm

Provide a plugin similar to Sentry's Redux integration

Initial MVP doesn;t need to cover all API surface of the redux integration

Potentially more to investigate in terms of SSR tracing within Nuxt apps.
@mydea mydea added Meta: Help Wanted Package: vue Issues related to the Sentry Vue SDK Package: nuxt Issues related to the Sentry Nuxt SDK labels Aug 8, 2024
@onurtemizkan onurtemizkan self-assigned this Sep 26, 2024
@onurtemizkan onurtemizkan mentioned this issue Oct 2, 2024
Merged
@s1gr1d s1gr1d closed this as completed in ecf84e0 Oct 15, 2024
@github-actions GitHub Actions
Copy link
Contributor

A PR closing this issue has just been released 🚀

This issue was referenced by PR #13841, which was included in the 8.35.0-beta.0 release.

billyvg pushed a commit that referenced this issue Oct 17, 2024
@onurtemizkan @billyvg
feat(vue): Add Pinia plugin (#13841)
1877473 
Resolves: #13279
Depends on: #13840 
[Sample
Event](https://sentry-sdks.sentry.io/issues/5939879614/?project=5429219&query=is%3Aunresolved%20issue.priority%3A%5Bhigh%2C%20medium%5D&referrer=issue-stream&sort=date&statsPeriod=1h&stream_index=0)

Docs PR: getsentry/sentry-docs#11516
 

Adds a Pinia plugin with a feature set similar to the Redux integration.

- Attaches Pinia state as an attachment to the event (`true` by default)
- Provides `actionTransformer` and `stateTransformer` to the user for
potentially required PII modifications.
- Adds breadcrumbs for Pinia actions
- Assigns Pinia state to event contexts.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@onurtemizkan onurtemizkan

Labels
Package: nuxt Issues related to the Sentry Nuxt SDK Package: vue Issues related to the Sentry Vue SDK
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(vue): Add Pinia plugin getsentry/sentry-javascript
3 participants
@mydea @onurtemizkan @smeubank