Skip to content
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

FileResource implementation #66

Merged
merged 11 commits into from
May 5, 2020
Merged

FileResource implementation #66

merged 11 commits into from
May 5, 2020

Conversation

Alex009
Copy link
Member

@Alex009 Alex009 commented May 4, 2020

  • files directory in MR for files
  • on Android used raw resources for storage of files
  • on iOS all files contains in files directory of bundle

OS differences

if we use same filename in different modules - on android will be used one of this files (from parent module), on iOS will be used different files.
if we have:

  • mpp-library/src/commonMain/resources/MR/files/test.txt with first contend
  • mpp-library/nested/src/commonMain/resources/MR/files/test.txt with second content
    on android if we read mpplib.MR.files.test we got first and for mpplib.nested.MR.files.test too will be first
    but on iOS will be as expected: mpplib.MR.files.test = first, mpplib.nested.MR.files.test = second

how it solve i don't know now

@Alex009 Alex009 added this to the 0.10.0 milestone May 4, 2020
@Alex009 Alex009 requested a review from Tetraquark May 4, 2020 03:49
@Alex009 Alex009 self-assigned this May 4, 2020
@Alex009 Alex009 linked an issue May 4, 2020 that may be closed by this pull request
@Alex009 Alex009 mentioned this pull request May 4, 2020
# Conflicts:
#	gradle-plugin/src/main/kotlin/dev/icerock/gradle/generator/PluralsGenerator.kt
#	gradle-plugin/src/main/kotlin/dev/icerock/gradle/generator/StringsGenerator.kt
@Alex009 Alex009 requested a review from Tetraquark May 4, 2020 13:10
@Alex009 Alex009 merged commit 116cd45 into develop May 5, 2020
@Alex009 Alex009 deleted the #44-file-resource branch May 5, 2020 02:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Shared file resources
2 participants