Add Lint Warning or Deprecate usages of <deeplink> inside <activity> tag. [178403185]

Fixed

Feature Request

androidx-bug-bounty

Status Update

No update yet.

Description

fr...@gmail.com

created issue #1

Jan 26, 2021 02:53PM

Component used: Navigation Version used: 2.3.2

It has been recommended that using <deeplink> inside an <activity> tag on navigation graph should not be used as it has unintended side effects (as per this issue - https://issuetracker.google.com/issues/174934674). Would it be possible to deprecate this feature so that new developers are also aware of this issue? Or a lint warning would also help convey that this is not the correct approach going forward.

Comments

il...@google.com <il...@google.com> #2Jan 26, 2021 08:57PM

Assigned to jb...@google.com.

This has been fixed internally in

https://android-review.googlesource.com/870975 and will be available in Navigation 1.0.0-alpha10.

Now, instead of crashing, we ignore invalid deep links, printing a log message saying as much.

as...@google.com <as...@google.com> Apr 15, 2022 02:36PM

Reassigned to ju...@google.com.

jb...@google.com <jb...@google.com> #3Apr 15, 2022 09:35PM

Hi team.
The problem still occurs on version 2.2.0-rc04

as...@google.com <as...@google.com> #4Apr 18, 2022 01:22PM

Hi, still getting crash for 2.1.0

ap...@google.com <ap...@google.com> #5Apr 22, 2022 02:17PM

still getting crash on version 2.2.0

il...@google.com <il...@google.com> Apr 25, 2022 10:35PM

Marked as fixed.

Issue 178403185