-
Notifications
You must be signed in to change notification settings - Fork 114
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
Release R152 #1368
Release R152 #1368
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
From the point of view of the loaders, I don't see any problems.
I noticed that cross_navigation_config
is the only schema without descriptions on the fields.
And I noticed that some files use tabs and some use spaces for whitespace. In case you care about that type of thing. (FWIW 189 out of 537 schemas in Iglu Central use tabs).
163b6b2
to
d0423c7
Compare
Thanks @istreeter! I have now converted the indentation to 2 spaces in all schemas and added field descriptions in |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pretty much just questions for my understanding
schemas/com.snowplowanalytics.snowplow/cross_navigation/jsonschema/1-0-0
Outdated
Show resolved
Hide resolved
schemas/com.snowplowanalytics.snowplow/cross_navigation/jsonschema/1-0-0
Show resolved
Hide resolved
schemas/com.snowplowanalytics.snowplow/cross_navigation/jsonschema/1-0-0
Show resolved
Hide resolved
schemas/com.snowplowanalytics.snowplow/cross_navigation/jsonschema/1-0-0
Outdated
Show resolved
Hide resolved
d0423c7
to
b70fe3b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
schemas/com.snowplowanalytics.snowplow/cross_navigation/jsonschema/1-0-0
Outdated
Show resolved
Hide resolved
schemas/com.snowplowanalytics.snowplow/cross_navigation/jsonschema/1-0-0
Outdated
Show resolved
Hide resolved
…g/jsonschema/1-0-0
b70fe3b
to
5e025f1
Compare
This release contains only new schemas.
visionOS events and entities (@mscwilson)
PR #1362
Add com.apple.swiftui/dismiss_immersive_space/jsonschema/1-0-0 (#1367)
Add com.apple.swiftui/open_immersive_space/jsonschema/1-0-0 (#1367)
Add com.apple.swiftui/dismiss_window/jsonschema/1-0-0 (#1367)
Add com.apple.swiftui/open_window/jsonschema/1-0-0 (#1367)
Add com.apple.swiftui/immersive_space/jsonschema/1-0-0 (#1367)
Add com.apple.swiftui/window_group/jsonschema/1-0-0 (#1367)
Mobile screen engagement tracking (@matus-tomlein)
PR #1358
Add com.snowplowanalytics.mobile/scroll_changed/jsonschema/1-0-0 (#1361)
Add com.snowplowanalytics.mobile/list_item_view/jsonschema/1-0-0 (#1360)
Add com.snowplowanalytics.mobile/screen_end/jsonschema/1-0-0 (#1357)
Add com.snowplowanalytics.mobile/screen_summary/jsonschema/1-0-0 (#1356)
Cross-navigation entity and config schema (@igneel64 and @adatzer)
PR #1347 and #1327
Add com.snowplowanalytics.snowplow.enrichments/cross_navigation_config/jsonschema/1-0-0
Add com.snowplowanalytics.snowplow/cross_navigation/jsonschema/1-0-0 (#1326)