-
-
Notifications
You must be signed in to change notification settings - Fork 377
Open
Description
Note
The pull request "refactor: Update SentryAttribute to use a generic array type" was created by @itaybre but did not reference an issue. Therefore this issue was created for better visibility in external tools like Linear.
📜 Description
Replaces the array attributes type with a generic array
💡 Motivation and Context
After reviewing the developer docs, I found out that array attributes now expect array as the type.
Related:
- Docs fix: Add example of
arrayattribute to telemetry attributes sentry-docs#15994 - Change in Sentry Native fix: update array attribute types to general 'array' string sentry-native#1479
- Relay feat(eap): Add support for native scalar arrays relay#5394
💚 How did you test it?
For now only unit tests, array attributes are not visible in Sentry UI so only unit tests
📝 Checklist
You have to check all boxes before merging:
- I added tests to verify the changes.
- No new PII added or SDK only sends newly added PII if
sendDefaultPIIis enabled. - I updated the docs if needed.
- I updated the wizard if needed.
- Review from the native team if needed.
- No breaking change or entry added to the changelog.
- No breaking change for hybrid SDKs or communicated to hybrid SDKs.
Metadata
Metadata
Assignees
Labels
No labels