Skip to content

Commit

Permalink
VendorId should be a String (#1112)
Browse files Browse the repository at this point in the history
  • Loading branch information
marandaneto authored Nov 16, 2022
1 parent 379d7a8 commit 453e1bc
Show file tree
Hide file tree
Showing 4 changed files with 11 additions and 7 deletions.
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@

### Fixes

- VendorId should be a String ([#1112](https://github.com/getsentry/sentry-dart/pull/1112))
- Disable `enableUserInteractionBreadcrumbs` on Android when `enableAutoNativeBreadcrumbs` is disabled ([#1131](https://github.com/getsentry/sentry-dart/pull/1131))
- Transaction name is reset after the transaction finishes ([#1125](https://github.com/getsentry/sentry-dart/pull/1125))

Expand All @@ -12,6 +13,9 @@
- Bump Cocoa SDK from v7.30.2 to v7.31.0 ([#1132](https://github.com/getsentry/sentry-dart/pull/1132))
- [changelog](https://github.com/getsentry/sentry-cocoa/blob/master/CHANGELOG.md#7310)
- [diff](https://github.com/getsentry/sentry-cocoa/compare/7.30.2...7.31.0)
- Bump Android SDK from v6.7.0 to v6.7.1 ([#1112](https://github.com/getsentry/sentry-dart/pull/1112))
- [changelog](https://github.com/getsentry/sentry-java/blob/main/CHANGELOG.md#671)
- [diff](https://github.com/getsentry/sentry-java/compare/6.7.0...6.7.1)

## 6.15.1

Expand Down
4 changes: 2 additions & 2 deletions dart/lib/src/protocol/sentry_gpu.dart
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ class SentryGpu {
final int? id;

/// The PCI vendor identifier of the graphics device.
final int? vendorId;
final String? vendorId;

/// The vendor name as reported by the graphics device.
final String? vendorName;
Expand Down Expand Up @@ -150,7 +150,7 @@ class SentryGpu {
SentryGpu copyWith({
String? name,
int? id,
int? vendorId,
String? vendorId,
String? vendorName,
int? memorySize,
String? apiType,
Expand Down
8 changes: 4 additions & 4 deletions dart/test/protocol/sentry_gpu_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ void main() {
final sentryGpu = SentryGpu(
name: 'fixture-name',
id: 1,
vendorId: 2,
vendorId: '2',
vendorName: 'fixture-vendorName',
memorySize: 3,
apiType: 'fixture-apiType',
Expand All @@ -17,7 +17,7 @@ void main() {
final sentryGpuJson = <String, dynamic>{
'name': 'fixture-name',
'id': 1,
'vendor_id': 2,
'vendor_id': '2',
'vendor_name': 'fixture-vendorName',
'memory_size': 3,
'api_type': 'fixture-apiType',
Expand Down Expand Up @@ -63,7 +63,7 @@ void main() {
final copy = data.copyWith(
name: 'name1',
id: 11,
vendorId: 22,
vendorId: '22',
vendorName: 'vendorName1',
memorySize: 33,
apiType: 'apiType1',
Expand All @@ -74,7 +74,7 @@ void main() {

expect('name1', copy.name);
expect(11, copy.id);
expect(22, copy.vendorId);
expect('22', copy.vendorId);
expect('vendorName1', copy.vendorName);
expect(33, copy.memorySize);
expect('apiType1', copy.apiType);
Expand Down
2 changes: 1 addition & 1 deletion flutter/android/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,6 @@ android {
}

dependencies {
api 'io.sentry:sentry-android:6.7.0'
api 'io.sentry:sentry-android:6.7.1'
implementation "org.jetbrains.kotlin:kotlin-stdlib-jdk8:$kotlin_version"
}

0 comments on commit 453e1bc

Please sign in to comment.