Remove unexpected quote escaping in string resource

[SimonMarquis]

[martinbonnin]

👍

[SimonMarquis]

Oh, interesting.

/home/runner/work/Confetti/Confetti/androidApp/src/main/res/values/strings.xml:36:4: Failed to flatten XML for resource 'sign_in_for_bookmarks' with error: Invalid unicode escape sequence in string
"{str}"

Not sure what's wrong. Wrapping in double quotes would probably fix the issue. But testing this is kinda tricky on a smartphone 😅

[martinbonnin]

Looks like this one works without the \ but the other one needs it...

[SimonMarquis]

@martinbonnin @joreilly here are some related issues:

• Support escaping special characters in string resources files JetBrains/compose-multiplatform#4178
• Escape char not supported on strings.xml JetBrains/compose-multiplatform#4664

that brings up the question of why are we trying to mirror string resources in androidApp/src/main/res/values/strings.xml since they don't override the ones defined in shared/src/commonMain/composeResources/values/strings.xml?

It seems like resources that are directly used in the shared module can't be modified by regular Android string resources. What do you think about declaring those only in the shared module (with the -fr alternatives)?

<resources>
    <string name="sessions">Sessions</string>
    <string name="oops">Oops something went wrong</string>
    <string name="retry">Retry</string>
    <string name="sign_in_for_bookmarks">In order to save your bookmarks, you'll need to sign in.</string>
    <string name="cancel">CANCEL</string>
    <string name="sign_in">SIGN IN</string>
</resources>

[joreilly]

Hmm, I think I might have been in process of transitioning those strings to shared code which is why they're probably still duplicated. Will take a look soon at cleaning that up.