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

Update vendor and version of the player schema used in the media plugin (close #1222) #1223

Merged
merged 3 commits into from
Aug 9, 2023

Conversation

matus-tomlein
Copy link
Contributor

Issue #1222

This PR updates the schema for the player entity used in the media plugin.

The previous schema had a conflict with another one which could lead to problems when loading data to warehouses. To face this, we republished the schema under the URI iglu:com.snowplowanalytics.snowplow/media_player/jsonschema/2-0-0.

@bundlemon
Copy link

bundlemon bot commented Aug 8, 2023

BundleMon

Files added (6)
Status Path Size Limits
libraries/browser-tracker-core/dist/index.mod
ule.js
+25.07KB 26KB / +10%
trackers/javascript-tracker/dist/sp.js
+24.03KB 25KB / +10%
trackers/javascript-tracker/dist/sp.lite.js
+14.7KB 15KB / +10%
trackers/browser-tracker/dist/index.umd.min.j
s
+14.55KB 15KB / +10%
libraries/tracker-core/dist/index.module.js
+13.35KB 15KB / +10%
trackers/browser-tracker/dist/index.module.js
+3.46KB 5KB / +10%

Total files change +95.15KB 0%

Final result: ✅

View report in BundleMon website ➡️


Current branch size history

Copy link
Contributor

@igneel64 igneel64 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Goo 📺

@@ -2,7 +2,7 @@ import { MediaEventType } from './types';

const MEDIA_SCHEMA_PREFIX = 'iglu:com.snowplowanalytics.snowplow.media/';
const MEDIA_SCHEMA_SUFFIX = '/jsonschema/1-0-0';
export const MEDIA_PLAYER_SCHEMA = MEDIA_SCHEMA_PREFIX + 'player' + MEDIA_SCHEMA_SUFFIX;
export const MEDIA_PLAYER_SCHEMA = 'iglu:com.snowplowanalytics.snowplow/media_player/jsonschema/2-0-0';
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We might want to leave a comment thought, WDYT ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good thinking, have added a comment!

@matus-tomlein matus-tomlein changed the title Update version of the player schema used in the media plugin to 2-0-0 (close #1222) Update vendor and version of the player schema used in the media plugin (close #1222) Aug 8, 2023
Copy link
Contributor

@greg-el greg-el left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 🚢

@matus-tomlein matus-tomlein merged commit ed9bbba into release/3.14.0 Aug 9, 2023
3 checks passed
@matus-tomlein matus-tomlein deleted the issue/1222-update_player_schema branch August 9, 2023 09:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants