-
Notifications
You must be signed in to change notification settings - Fork 62
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
Documentation of Single APK Approach Configuration #1313
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1313 +/- ##
=========================================
Coverage 60.00% 60.00%
Complexity 687 687
=========================================
Files 135 135
Lines 4691 4691
Branches 736 736
=========================================
Hits 2815 2815
- Misses 1455 1456 +1
+ Partials 421 420 -1
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
This looks good @FikriMilano . I'll give further comments later in the day . Good job |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I want to review text before merge, blocking merge for the moment
* main: Documentation of Single APK Approach Configuration (#1313)
* Create configuration.md * Rename docs/configuration.md to docs/configurability/readme.md * Update and rename app-configurability.md to bottom-nav-config.md * Update readme.md Co-authored-by: Benjamin Mwalimu <dubdabasoduba@gmail.com> Co-authored-by: Peter Lubell-Doughtie <peter@ona.io>
IMPORTANT: Where possible all PRs must be linked to a Github issue
Fixes #1284
To view the documentation in a non-raw form, open this link
https://github.com/opensrp/fhircore/blob/documenting-configurable-app/docs/configuration.md
Checklist
strings.xml
file./gradlew spotlessApply
and./gradlew spotlessCheck
to check my code follows the project's style guide