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

Patch #18

Closed
wants to merge 3 commits into from
Closed

Patch #18

wants to merge 3 commits into from

Conversation

shahin68
Copy link
Owner

@shahin68 shahin68 commented Jun 25, 2024

Extract string resource related to feature module "feature"book_details

  • This string can be provided by the feature module itself since it's heavily relevant to the feature module and not considered a widely used string

  • However, for better maintenance it's advised all resources to be provided by the main app when possible. But for this one, considering the feature module location is also valid

Sort books by release date provided

Sort by Id if : Sort by id lastly in ascending order if other sorting fail

shahin68 added 3 commits June 25, 2024 02:50
…ils`

- This string can be provided by the feature module itself since it's heavily relevant to the feature module and not considered a widely used string

- However, for better maintenance it's advised all resources to be provided by the main app when possible. But for this one, considering the feature module location is also valid
@shahin68 shahin68 self-assigned this Jun 25, 2024
@shahin68
Copy link
Owner Author

After digging though sqlcipher-android Failure is caused due to passphrase miss matching for some reason.
But seems not even sqlcipher team have enough information regarding this issue.
Based on the current state, this PR isn't fit to be merged into master branch until the issue is either resolved or reworked.

References:
sqlcipher/sqlcipher-android#32
sqlcipher/sqlcipher-android#22
sqlcipher/sqlcipher-android#34

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.

1 participant