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

Trunk-3850 #664

Closed
wants to merge 1 commit into from
Closed

Trunk-3850 #664

wants to merge 1 commit into from

Conversation

suraj5
Copy link

@suraj5 suraj5 commented Feb 24, 2014

No description provided.

@ghost
Copy link

ghost commented Mar 6, 2014

Please use more a descriptive pull request title, similar to that as described for commit messages in https://wiki.openmrs.org/display/docs/Using+Git under step 3 in section "Work on your task".

@dkayiwa
Copy link
Member

dkayiwa commented Mar 7, 2014

@suraj5 can you also do a second pull request for master? Please always first run "mvn clean install" before committing to ensure that all unit tests pass and that your changes get formatted according to our conventions.

@suraj5
Copy link
Author

suraj5 commented Mar 8, 2014

@dkayiwa trunk-1748 by @mseaton .In the comments section you can see mike seaton last comment ,he said that the 'ProgramWorkflowService.getProgramByName()' should look at the program name, not the concept name.

@suraj5 suraj5 closed this Mar 8, 2014
@suraj5 suraj5 deleted the trunk-3850 branch March 8, 2014 12:13
RandilaP pushed a commit to RandilaP/openmrs-core that referenced this pull request Jul 31, 2023
…penmrs#664)

* Fixed the redirection after selecting no location

* Review changes
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