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

fix(SUP-37445): Identifying tags that include 360 as 360 videos #212

Merged
merged 1 commit into from
May 30, 2023

Conversation

Tzipi-kaltura
Copy link
Contributor

Description of the Changes

the issue:
when tag contains space and 360 it's activate the vr. the expected behavior is only if the tag is exact match to 360 the vr will be activated.

the solution:
check exact match to 360

CheckLists

  • changes have been done against master branch, and PR does not conflict
  • new unit / functional tests have been added (whenever applicable)
  • test are passing in local environment
  • Travis tests are passing (or test results are not worse than on master branch :))
  • Docs have been updated

@Tzipi-kaltura Tzipi-kaltura merged commit 01cf5d8 into master May 30, 2023
@Tzipi-kaltura Tzipi-kaltura deleted the SUP-37445 branch May 30, 2023 06:53
giladna added a commit that referenced this pull request Jun 8, 2023
* master: (40 commits)
  chore(release): 2.39.4
  fix(FEC-13134): add ceratedAt and endedDate to media entry metadata if available (#216)
  fix(SUP-37445): Identifying tags that include 360 as 360 videos (#212)
  fix(FEC-13123): replace travis with github actions yarn add exact (#215)
  chore: trigger build
  fix: trigger build
  fix(FEC-13146): Kava sent the View event 4 times - fix POST method (#214)
  fix(FEC-13146): Kava sent the View event 4 times (#213)
  chore(release): 2.39.3
  fix(FEC-13123): replace travis with github actions (#210)
  fix(FEC-13123): replace travis with github actions (#209)
  chore(release): 2.39.2
  add gitignore
  chore(release): 2.39.1
  fix(FEC-13123): replace travis with github actions (#208)
  chore(FEC-13123): replace travis with github actions (#207)
  chore: update dist
  fix(FEC-13113): fix and add unit tests in handleRegexAction file (#206)
  chore(release): 2.39.0
  chore: update dist
  ...

# Conflicts:
#	test/src/k-provider/ott/media-config-data.js
giladna added a commit that referenced this pull request Jun 8, 2023
* master:
  chore(release): 2.39.4
  fix(FEC-13134): add ceratedAt and endedDate to media entry metadata if available (#216)
  fix(SUP-37445): Identifying tags that include 360 as 360 videos (#212)

# Conflicts:
#	src/k-provider/ovp/provider-parser.js
#	src/k-provider/ovp/request-params/base-entry-response-profile.js
#	src/k-provider/ovp/response-types/kaltura-media-entry.js
giladna added a commit that referenced this pull request Jun 19, 2023
* master:
  fix(FEC-13134): fix playlist incorrect media type is it was live and became vod (#217)
  chore(release): 2.39.4
  fix(FEC-13134): add ceratedAt and endedDate to media entry metadata if available (#216)
  fix(SUP-37445): Identifying tags that include 360 as 360 videos (#212)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants