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

Updating concepts to CIEL 2.1_20190227 #42

Merged
merged 1 commit into from
Mar 19, 2019

Conversation

reubenvarghese1
Copy link
Contributor

Updating concepts to CIEL 2.1_20190227

@reubenvarghese1
Copy link
Contributor Author

@dkayiwa, these are all the significant changes made as part of the latest ciel release

@dkayiwa
Copy link
Member

dkayiwa commented Mar 10, 2019

They do not look that many. Are they?

@reubenvarghese1
Copy link
Contributor Author

There were over 2000 changes but were all minor changes like date changes etc. As you had mentioned in the previous pr reviews, I ignored these changes. Finally what remains is this

@reubenvarghese1
Copy link
Contributor Author

@dkayiwa , that being said , I'm more worried about what I've mentioned here : https://talk.openmrs.org/t/reference-application-2-9-testing/21389/44?u=reubenv

@reubenvarghese1
Copy link
Contributor Author

@dkayiwa I have squashed the previous commits and ensured that 165095 is added back. Thanks to @rkorytkowski who solved it by adding it individually as per this post

@dkayiwa dkayiwa merged commit 2cdd57b into openmrs:master Mar 19, 2019
@dkayiwa
Copy link
Member

dkayiwa commented Mar 20, 2019

@reubenvarghese1 did you test by running the module and confirm that the concept shows up?

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