# Working on ticket LUI-80

**URL:** https://talk.openmrs.org/t/working-on-ticket-lui-80/22966
**Category:** Development
**Created:** [April 24, 2019, 12:28pm UTC](https://talk.openmrs.org/t/working-on-ticket-lui-80/22966 "2019-04-24T12:28:14Z")
**Posts on this page:** 5
**Page:** 1

<div class="post-metadata">

### Author: ![odorajonathan](https://talk.openmrs.org/user_avatar/talk.openmrs.org/odorajonathan/32/8251_2.png) [@odorajonathan](https://talk.openmrs.org/u/odorajonathan)
#### Post date: [April 24, 2019, 12:28pm UTC](https://talk.openmrs.org/t/working-on-ticket-lui-80/22966/1 "2019-04-24T12:28:15Z")

</div>

Hi,

So am trying to make some changes for this [ticket](https://issues.openmrs.org/browse/LUI-80), am also running the changes on the RefApp but what’s holding me back is the fact that the code changes am making for it are not reflected as required by issue description on the RefApp

The changes am making are in the directory [https://github.com/openmrs/openmrs-module-legacyui/blob/master/omod/src/main/java/org/openmrs/web/controller/OptionsFormController.java](https://github.com/openmrs/openmrs-module-legacyui/blob/master/omod/src/main/java/org/openmrs/web/controller/OptionsFormController.java)

So what am not sure of is whether my code changes are effectively right or am not doing the right code changes, Really holding me back especially trying to remove one of the password fields in the `/openmrs/options.form` for the ReffApp. I am successful in adding the error message though.

@dkayiwa @herbert24 @mozzy

---

<div class="post-metadata">

### Author: ![dkayiwa](https://talk.openmrs.org/user_avatar/talk.openmrs.org/dkayiwa/32/179_2.png) [@dkayiwa](https://talk.openmrs.org/u/dkayiwa)
#### Post date: [April 24, 2019, 12:35pm UTC](https://talk.openmrs.org/t/working-on-ticket-lui-80/22966/2 "2019-04-24T12:35:00Z")

</div>

Can you raise a pull request for the code changes that you are making and then we review to evaluate of they should have any effects?

---

<div class="post-metadata">

### Author: ![mozzy](https://talk.openmrs.org/user_avatar/talk.openmrs.org/mozzy/32/11815_2.png) [@mozzy](https://talk.openmrs.org/u/mozzy)
#### Post date: [April 24, 2019, 12:40pm UTC](https://talk.openmrs.org/t/working-on-ticket-lui-80/22966/3 "2019-04-24T12:40:47Z")

</div>

@odorajonathan .thanks for the work your doing. to add on what @dkayiwa has said, whenever a ticket requires a UI change, you always must be able to see the UI change locally . that is if you made the right code changes

---

<div class="post-metadata">

### Author: ![mozzy](https://talk.openmrs.org/user_avatar/talk.openmrs.org/mozzy/32/11815_2.png) [@mozzy](https://talk.openmrs.org/u/mozzy)
#### Post date: [April 24, 2019, 12:42pm UTC](https://talk.openmrs.org/t/working-on-ticket-lui-80/22966/4 "2019-04-24T12:42:33Z")

</div>

> [@odorajonathan](#):
>
> The changes am making are in the directory [https://github.com/openmrs/openmrs-module-legacyui/blob/master/omod/src/main/java/org/openmrs/web/controller/OptionsFormController.java](https://github.com/openmrs/openmrs-module-legacyui/blob/master/omod/src/main/java/org/openmrs/web/controller/OptionsFormController.java)

but also , if you’re required to make some UI changes , i would expect you to have certain changes in some jsp file, some how some where on top of having a change in the controller class

---

<div class="post-metadata">

### Author: ![odorajonathan](https://talk.openmrs.org/user_avatar/talk.openmrs.org/odorajonathan/32/8251_2.png) [@odorajonathan](https://talk.openmrs.org/u/odorajonathan)
#### Post date: [April 24, 2019, 12:53pm UTC](https://talk.openmrs.org/t/working-on-ticket-lui-80/22966/5 "2019-04-24T12:53:23Z")

</div>

@mozzy Thanks for the heads up, let me update the PR so that I can easily get to know which jsp files need a change as well in addition
