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

Thrivepk branch for merge to master #296

Closed
wants to merge 64 commits into from
Closed

Thrivepk branch for merge to master #296

wants to merge 64 commits into from

Conversation

maimoonak
Copy link

#295 Addressing issue of core merging of vaccinator app to master

Maimoona Kausar added 30 commits May 30, 2016 15:03
test server
Conflicts:
	opensrp-app/res/xml/preferences.xml
Conflicts:
	opensrp-app/res/drawable-mdpi/homestacks.png
	opensrp-app/res/layout/smart_register_status_bar.xml
	opensrp-app/src/main/java/org/ei/opensrp/view/fragment/SecuredNativeSmartRegisterFragment.java
	opensrp-vaccinator/src/main/java/org/ei/opensrp/vaccinator/child/ChildSmartRegisterActivity.java
	opensrp-vaccinator/src/main/java/org/ei/opensrp/vaccinator/field/FieldMonitorSmartRegisterActivity.java
	opensrp-vaccinator/src/main/java/org/ei/opensrp/vaccinator/woman/WomanSmartRegisterActivity.java
Conflicts:
	opensrp-app/res/drawable-mdpi/homestacks.png
	opensrp-app/src/main/java/org/ei/opensrp/view/fragment/SecuredNativeSmartRegisterFragment.java
	opensrp-mcare/src/main/java/org/ei/opensrp/mcare/child/mCareChildServiceModeOption.java
	opensrp-mcare/src/main/res/layout/childdetail_nav_bar.xml
	opensrp-vaccinator/src/main/res/layout/childdetail_nav_bar.xml
	opensrp-vaccinator/src/main/res/layout/detail_nav_bar.xml
Conflicts:
	opensrp-vaccinator/src/main/java/org/ei/opensrp/vaccinator/child/ChildSmartClientsProvider.java
	opensrp-vaccinator/src/main/java/org/ei/opensrp/vaccinator/woman/WomanSmartClientsProvider.java
	opensrp-vaccinator/src/main/java/util/Utils.java
… push in chunks, Fixed registers to allow invalid data

Conflicts:
	opensrp-app/assets/web/forms/footer
	opensrp-vaccinator/build.gradle
	opensrp-vaccinator/src/main/assets/www/form/child_enrollment/form.json
	opensrp-vaccinator/src/main/assets/www/form/child_enrollment/form.xml
	opensrp-vaccinator/src/main/assets/www/form/child_followup/form.json
	opensrp-vaccinator/src/main/assets/www/form/child_followup/form.xml
	opensrp-vaccinator/src/main/assets/www/form/offsite_child_followup/form.json
	opensrp-vaccinator/src/main/assets/www/form/offsite_child_followup/form.xml
	opensrp-vaccinator/src/main/assets/www/form/offsite_woman_followup/form.json
	opensrp-vaccinator/src/main/assets/www/form/offsite_woman_followup/form.xml
	opensrp-vaccinator/src/main/assets/www/form/offsite_woman_followup/form_definition.json
	opensrp-vaccinator/src/main/assets/www/form/woman_enrollment/form.json
	opensrp-vaccinator/src/main/assets/www/form/woman_followup/form.json
	opensrp-vaccinator/src/main/assets/www/form/woman_followup/form.xml
	opensrp-vaccinator/src/main/java/org/ei/opensrp/vaccinator/child/ChildDetailActivity.java
	opensrp-vaccinator/src/main/java/org/ei/opensrp/vaccinator/woman/DetailActivity.java
	opensrp-vaccinator/src/main/java/org/ei/opensrp/vaccinator/woman/WomanDetailActivity.java
…n all registers made a little more padded, Changed version to 1.2.0, Issues targeted #221 - #225
…Removed required validation from retro vaccines, changed version to 1.2.3
Min version 18 and compile version 23 target version 21.
Build tool version updated to 23.0.2.
Picasso dependency moved to core app from vaccinator app.
…uired for vaccinator app. Site should add it to module lib if needed. (JakeWharton/butterknife#100)
…ServiceModes, SortOptions, FilterOptions, and Layouts and Views which were register specific and should have been in site module rather than core. These were inherited from drishti app. All tests spcific to those also removed from core.
…for Fragment and Activity both. Also launches playstorre for QR code app installation if not exists
…s but details so all data for client in columns is accessible via columnmap. Also changed to index to column name so that change of column index doesnot impacct
Maimoona Kausar added 20 commits July 14, 2016 14:20
…oman register also added wih new grid column that is contact number
… since fragment already loads all views and adapter data.
…instead of calling it on view load. Fixed monthly reports as well
… core activity which provides syncers, UI rendering and child provides site specific details.
@raihan-mpower
Copy link

this is a huge pull request , my initial observation is that there are lot of changes like css and resource change stuff done in core library . It would be great if @keymane or @maina could look at it and also if you could achieve that by overriding the resources of the library in your module instead of opensrp-client module . For references maybe you can look into these links (i am sure there are way better articles on overriding resources/resource merging or resource referencing between libraries and project modules):
https://developer.android.com/studio/write/add-resources.html
http://tools.android.com/tech-docs/new-build-system/resource-merging

@maimoonak
Copy link
Author

@maimoonak
Copy link
Author

Thanks @raihan-mpower . This is helpful. I should have overridden the enkteo css and js in project. Other resources have been deleted and modified intentionally. I would explain it in detail in call this week.

@raihan-mpower
Copy link

7c135c4
this would require us to change a lot of code , infact declare all columns in bindtypes.json file and also in all of our codes in across all three apps. If you want to use this you can extend commonRepository and override the methods (Probably name it ColumnedCommonRepository). Then use the extended one in your registers. That way it would probably not break the current structure. Else this is going to create a lot of problems in all our apps if we were to merge this with the master.

@maimoonak
Copy link
Author

maimoonak commented Jan 8, 2017 via email

@maimoonak
Copy link
Author

Not applicable

@maimoonak maimoonak closed this Jan 18, 2017
@maimoonak maimoonak deleted the thrivepk branch January 18, 2017 04:02
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