Fix media controls notification showing disambiguation drawer #1238

Open
ktprograms wants to merge 1 commit from ktprograms/media-controls-notification-no-disambiguation into master
ktprograms commented 2022-05-23 10:37:34 +02:00 (Migrated from github.com)

PR Checklist

Please check all that apply to this PR using "x":

  • I have checked that this PR is not a duplicate of an existing PR (open, closed or merged)
  • I have checked that this PR does not introduce a breaking change

PR Type

What kind of change does this PR introduce?

  • Bugfix

Fixes

Issue Number: N/A

Backport of fix in https://github.com/OdyseeTeam/odysee-android/pull/212

What is the current behavior?

Clicking media controls notification show disambiguation drawer with LBRY and Odysee apps available.

What is the new behavior?

Directly open Odysee MainActivity when clicking media controls notification.

Use direct intent to MainActivity instead of deep link intent.

## PR Checklist <!-- For the checkbox formatting to work properly, make sure there are no spaces on either side of the "x" --> Please check all that apply to this PR using "x": - [x] I have checked that this PR is not a duplicate of an existing PR (open, closed or merged) - [x] I have checked that this PR does not introduce a breaking change ## PR Type What kind of change does this PR introduce? - [x] Bugfix ## Fixes Issue Number: N/A Backport of fix in https://github.com/OdyseeTeam/odysee-android/pull/212 ## What is the current behavior? Clicking media controls notification show disambiguation drawer with LBRY and Odysee apps available. ## What is the new behavior? Directly open Odysee MainActivity when clicking media controls notification. Use direct intent to MainActivity instead of deep link intent.
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin ktprograms/media-controls-notification-no-disambiguation:ktprograms/media-controls-notification-no-disambiguation
git checkout ktprograms/media-controls-notification-no-disambiguation

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git checkout master
git merge --no-ff ktprograms/media-controls-notification-no-disambiguation
git checkout ktprograms/media-controls-notification-no-disambiguation
git rebase master
git checkout master
git merge --ff-only ktprograms/media-controls-notification-no-disambiguation
git checkout ktprograms/media-controls-notification-no-disambiguation
git rebase master
git checkout master
git merge --no-ff ktprograms/media-controls-notification-no-disambiguation
git checkout master
git merge --squash ktprograms/media-controls-notification-no-disambiguation
git checkout master
git merge --ff-only ktprograms/media-controls-notification-no-disambiguation
git checkout master
git merge ktprograms/media-controls-notification-no-disambiguation
git push origin master
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: LBRYCommunity/lbry-android#1238
No description provided.