[ 
http://mifosforge.jira.com/browse/MIFOS-4841?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=65983#comment-65983
 ] 

keithwoodlock commented on MIFOS-4841:
--------------------------------------

Hi Ashish,

Some feedback on 'define new system user functionality using spring webflow and 
freemarker'

1) I see you added in the properties into the savings_messages.properties file, 
seen as the rest of the properties used in freemaker pages fro user 
functioanlity are in 'messages.properties' i'll take then from savings_messages 
and put into messages to keep them together.

d)We need to match existing functionality some of which is not at first obvious 
but i did notice the following straightaway

 - preferred language is now dropped from define system user functionality but 
still exists on freemarker version
 - i couldn't actually step through and create a new user, an exception 
happened at the end etc

For now I won't worry to much about the exception occuring, having looked 
through the implementation (which was written some time ago) I can see that it 
can be done in a cleaner way so we should focus on going through each step in 
the 'flow' and cleaning up both the freemarker implmentation and the spring web 
flow implementation (user.xml file and associated form beans and DTOs use

To begin with 'choose office step'

need to clean up freemarker implementation.
- be consistent in how we display the wizard title and flow step title as with 
other flows like savings and loan
- remove use of tables for display/centering
- ensure we don't hard code anything into forms action all forms using webflow 
should look like this.
    <form action="${flowExecutionUrl}" ...
-  be consistent in how we display buttons/form controls on page.

To begin with i will make changes based on what i have said above. I will then 
give more feedback on implementation for you.

> UI Conversion: implement define new system user using freemarker and spring 
> weblfow
> -----------------------------------------------------------------------------------
>
>                 Key: MIFOS-4841
>                 URL: http://mifosforge.jira.com/browse/MIFOS-4841
>             Project: mifos
>          Issue Type: Story
>    Affects Versions: Release G
>            Reporter: keithwoodlock
>            Assignee: keithwoodlock
>             Fix For: Unscheduled
>
>


-- 
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

------------------------------------------------------------------------------
Free Software Download: Index, Search & Analyze Logs and other IT data in 
Real-Time with Splunk. Collect, index and harness all the fast moving IT data 
generated by your applications, servers and devices whether physical, virtual
or in the cloud. Deliver compliance at lower cost and gain new business 
insights. http://p.sf.net/sfu/splunk-dev2dev 
_______________________________________________
Mifos-issues mailing list
Mifos-issues@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mifos-issues

Reply via email to