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

changePlaybackPositionCommand を無効に #5

Merged
merged 5 commits into from
Feb 16, 2021

Conversation

xxxAIRINxxx
Copy link

経緯

Radiko の仕様的にCM飛ばしが出来る操作は許可しないとのことなので無効にしました。

https://radikohq.slack.com/archives/G01275MPKHV/p1612939992488700?thread_ts=1612938742.483200&cid=G01275MPKHV

@xxxAIRINxxx xxxAIRINxxx self-assigned this Feb 10, 2021
let changePlaybackPositionCommand = remoteCommandCenter!.changePlaybackPositionCommand
changePlaybackPositionCommand.isEnabled = true
changePlaybackPositionCommand.addTarget(handler: self.onChangePlaybackPositionCommand)
// Radiko では seek, skip はCMとのからみがあるので許可しないため無効にする
Copy link

Choose a reason for hiding this comment

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

念のため、#ifdefみたいなので機能無効にしておきたいですね

  • 仕様差分の把握を容易に
  • 将来的な仕様変更への備え

Copy link
Author

Choose a reason for hiding this comment

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

@reedom
ありがとうございます 🙇
enableChangePlaybackPosition を追加して対応することとしました 🙆‍♀️

9a8bc96

@reedom reedom merged commit 0529202 into master Feb 16, 2021
@reedom reedom deleted the disabled_change_playback_position branch February 16, 2021 02:21
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