Skip to content

929 simple base implementation#943

Merged
StaehliJ merged 12 commits into928-custom-castplayerfrom
929-simple-base-implementation
Mar 14, 2025
Merged

929 simple base implementation#943
StaehliJ merged 12 commits into928-custom-castplayerfrom
929-simple-base-implementation

Conversation

@StaehliJ
Copy link
Contributor

Pull request

Description

The first iteration to implement basic player features.

Changes made

  • New custom PillarboxCastPlayer with less feature that beforel.

Checklist

  • APIs have been properly documented (if relevant).
  • The documentation has been updated (if relevant).
  • New unit tests have been written (if relevant).
  • The demo has been updated (if relevant).

@StaehliJ StaehliJ linked an issue Mar 14, 2025 that may be closed by this pull request
7 tasks
@github-project-automation github-project-automation bot moved this to ✏️ Draft in Pillarbox Mar 14, 2025
set(value) {}
private val castPlayerListener = InternalCastPlayerListener()
private val mediaItemConverter: MediaItemConverter,
@IntRange(from = 1) private val seekBackIncrementMs: Long,

Check warning

Code scanning / detekt

Property is unused and should be removed. Warning

Private property seekBackIncrementMs is unused.
private val castPlayerListener = InternalCastPlayerListener()
private val mediaItemConverter: MediaItemConverter,
@IntRange(from = 1) private val seekBackIncrementMs: Long,
@IntRange(from = 1) private val seekForwardIncrementMs: Long,

Check warning

Code scanning / detekt

Property is unused and should be removed. Warning

Private property seekForwardIncrementMs is unused.
private val mediaItemConverter: MediaItemConverter,
@IntRange(from = 1) private val seekBackIncrementMs: Long,
@IntRange(from = 1) private val seekForwardIncrementMs: Long,
@IntRange(from = 0) private val maxSeekToPreviousPositionMs: Long,

Check warning

Code scanning / detekt

Property is unused and should be removed. Warning

Private property maxSeekToPreviousPositionMs is unused.
private val mediaQueueCallback = MediaQueueCallback()
private val analyticsCollector = DefaultAnalyticsCollector(clock).apply { addListener(EventLogger()) }

var sessionAvailabilityListener: SessionAvailabilityListener? = null

Check warning

Code scanning / detekt

Public properties require documentation. Warning

The property sessionAvailabilityListener is missing documentation.

var sessionAvailabilityListener: SessionAvailabilityListener? = null

var remoteMediaClient: RemoteMediaClient? = null

Check warning

Code scanning / detekt

Public properties require documentation. Warning

The property remoteMediaClient is missing documentation.
},
)

MainView(player = player, modifier = Modifier)

Check warning

Code scanning / Android Lint

Synthetic Accessor Warning

Access to private method MainView of class MainActivityKt requires synthetic accessor
@MGaetan89 MGaetan89 changed the base branch from main to 928-custom-castplayer March 14, 2025 09:35
@StaehliJ
Copy link
Contributor Author

We know that not every checks passes, it is a in progress tasks.

@StaehliJ StaehliJ merged commit 4fff31d into 928-custom-castplayer Mar 14, 2025
7 of 10 checks passed
@StaehliJ StaehliJ deleted the 929-simple-base-implementation branch March 14, 2025 10:47
@github-project-automation github-project-automation bot moved this from ✏️ Draft to ✅ Done in Pillarbox Mar 14, 2025
StaehliJ added a commit that referenced this pull request Apr 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

Simple base implementation with no playlist management

1 participant