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

https://tickets.openmrs.org/browse/TRUNK-15Trunk 15 #329

Closed
wants to merge 4 commits into from

Conversation

ellymakuba
Copy link

all comments are welcome to help me sort this ticket

@jlkeiper
Copy link
Member

jlkeiper commented Jun 3, 2013

I believe what you have provided is not what we are looking for. You provided two files: trunk-15 and TRUNK-15. They appear to be classes or code snippets, but would not be compiled into anything meaningful in the code.

I think you have the right idea but need to actually change the files you want to affect, compile the changes and possibly write some unit tests to support the ticket. Please ask in IRC, as you have been doing, and in the mailing list if you need more guidance. Thanks.

@jlkeiper jlkeiper closed this Jun 3, 2013
RandilaP pushed a commit to RandilaP/openmrs-core that referenced this pull request Jul 31, 2023
* (feat) Add options parameter to translateFrom

* Fixup
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants