Skip to content
This repository has been archived by the owner on May 6, 2024. It is now read-only.

chore: update chromecast for app nav #1811

Merged
merged 2 commits into from
Nov 21, 2023
Merged

chore: update chromecast for app nav #1811

merged 2 commits into from
Nov 21, 2023

Conversation

mumer92
Copy link
Contributor

@mumer92 mumer92 commented Nov 11, 2023

Description

LEARNER-9688

This PR updates Chromecast for the App Nav. Chromecast button is shown on video screen, and if casting is being done for a video of same course, CourseDashboard shows cast button.

Playing video on chromecast

RPReplay_Final1700027787.MP4

When chromecast is not available.

IMG_4104

Copy link
Contributor

@saeedbashir saeedbashir left a comment

Choose a reason for hiding this comment

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

The project is not running in Rosetta mode. Am I missing something?

image

Source/ChromeCastButtonDelegate.swift Outdated Show resolved Hide resolved
Source/CourseDashboardHeaderView.swift Outdated Show resolved Hide resolved
Source/ChromeCastButtonDelegate.swift Outdated Show resolved Hide resolved
edX.xcodeproj/project.pbxproj Show resolved Hide resolved
@saeedbashir
Copy link
Contributor

The project is not running in Rosetta mode. Am I missing something?

Figure out the issue now it's compiling at my end.

Copy link
Contributor

@saeedbashir saeedbashir left a comment

Choose a reason for hiding this comment

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

LGTM 👍
Wait for Hassan's testing and after that you will be good to merge.

@mumer92
Copy link
Contributor Author

mumer92 commented Nov 21, 2023

QA team was able to test this out on Chromecast, closing this PR after their approval.

@mumer92 mumer92 merged commit eeb90aa into master Nov 21, 2023
2 of 6 checks passed
@mumer92 mumer92 deleted the umer/LEARNER-9688 branch November 21, 2023 09:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants