Skip to content

Draft: chore(179): Share resources between iOS and Android #180

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

Merged
merged 10 commits into from
Feb 27, 2024

Conversation

enthuan
Copy link
Collaborator

@enthuan enthuan commented Feb 26, 2024

Added Moko Resources to the projet and shared iOS and Android font, images, colors and strings.

All the configuration is fine for me, I removed unused resources on Android and I started to share the strings.
Some strings are specific for iOS. I assume that they will be removed if we manage to go full Compose Multiplatform.

Copy link
Contributor

@martinbonnin martinbonnin left a comment

Choose a reason for hiding this comment

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

A few comments, definitely not blockers, feel free to merge 👍 . Pretty cool to be able to share strings like this 🤩

Comment on lines +1 to +6
<?xml version="1.0" encoding="UTF-8"?>
<Bucket
uuid = "9F7B89E4-5A4D-4E4C-A16E-817569E368B9"
type = "1"
version = "2.0">
</Bucket>
Copy link
Contributor

Choose a reason for hiding this comment

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

Should that be in .gitignore? (given it contains usernames in paths)

@enthuan enthuan merged commit 4e0c150 into paug:main Feb 27, 2024
@enthuan enthuan deleted the fix/179 branch February 27, 2024 10:54
martinbonnin pushed a commit that referenced this pull request Mar 7, 2025
* Added Moko Resources

* Configured Moko on iOS

* Plugged iOS strings to MP resources

* Removed unused resources

* Removed L10n from iOS


---------

Co-authored-by: Antoine Robiez <antoine.robiez@worldline.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants