-
Notifications
You must be signed in to change notification settings - Fork 991
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
MBL-1053: Identify if https links in messages are KSR links #1917
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #1917 +/- ##
=========================================
Coverage 74.29% 74.29%
Complexity 1963 1963
=========================================
Files 337 337
Lines 19679 19679
Branches 2730 2730
=========================================
Hits 14620 14620
Misses 3491 3491
Partials 1568 1568 ☔ View full report in Codecov by Sentry. |
@@ -48,7 +48,7 @@ interface MessageHolderViewModel { | |||
ActivityViewModel<MessageViewHolder?>(environment), | |||
Inputs, | |||
Outputs { | |||
private val currentUser: CurrentUserType? | |||
private val currentUser: CurrentUserTypeV2? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
very minor, but currentUser cant be null (due to requireNotNull(*)) so it doesn't need to be nullable here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll fix in a subsequent PR! Thanks!
📲 What
Goal is to validate links sent in messages by creators to open app links in-app where applicable.
🤔 Why
Previously, urls sent by creators would always open an external browser. Now, when it is a kickstarter link, it will open in app. All other links will continue to go to an external browser.
🛠 How
👀 See
messages-link-validation.mp4
📋 QA
Story 📖
MBL-1053: Identify if https links in messages are KSR links