-
Notifications
You must be signed in to change notification settings - Fork 5
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
chore: [ANDROAPP-6676] Updates compose, kotlin, gradle, and dependencies versions #340
Conversation
@@ -1,5 +1,6 @@ | |||
<?xml version="1.0" encoding="UTF-8" ?> | |||
<resources> | |||
<string name="no_string_default">No string fo this key</string> |
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.
small typo 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.
removed as its not needed
designsystem/src/commonMain/kotlin/org/hisp/dhis/mobile/ui/designsystem/component/AssistChip.kt
Show resolved
Hide resolved
@@ -79,6 +86,29 @@ fun SupportingText( | |||
LaunchedEffect(textLayoutResult) { | |||
if (textLayoutResult == null) return@LaunchedEffect | |||
|
|||
val link = LinkAnnotation.Clickable( |
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.
maybe this could be added to the ticket for things to be tested to verify that we haven't lost any functionality
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.
Snapshot tests have failed with the update the golden images have changed slightly, can you generate them again? that way we can see what the differences between them are
a8f69bb
to
f25fd35
Compare
No description provided.