Skip to content

#224 default error text#225

Open
ExNDY wants to merge 7 commits intoicerockdev:developfrom
ExNDY:#224-default-error-text
Open

#224 default error text#225
ExNDY wants to merge 7 commits intoicerockdev:developfrom
ExNDY:#224-default-error-text

Conversation

@ExNDY
Copy link
Contributor

@ExNDY ExNDY commented Feb 10, 2026

kotlin 2.1.0
moko-resources 0.26.0
reworked networkErrorTexts class

@ExNDY ExNDY self-assigned this Feb 10, 2026
@ExNDY ExNDY requested a review from Alex009 February 10, 2026 11:15
@ExNDY ExNDY added the enhancement New feature or request label Feb 10, 2026
import dev.icerock.moko.errors.MR as ErrorsMR

data class NetworkErrorsTexts(
val defaultErrorText: StringResource = ErrorsMR.strings.moko_errors_unknownError,
Copy link
Member

Choose a reason for hiding this comment

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

will be better if new argument of constructor will be latest. because now somewhere in exist codebase can be used positional version of constructor - with custom networkConnectionErrorText. in this case compiler will not see any errors, but in runtime behavior will be changed (instead of network connection error text will be changed default error text)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

mokoErrorsVersion = "0.9.0"
mokoTestVersion = "0.6.1"
mokoNetworkVersion = "0.22.0"
mokoNetworkVersion = "0.23.0"
Copy link
Member

Choose a reason for hiding this comment

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

readme should be updated too

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed, added Customizing error texts block

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants