Hi Nesrine, I apologize for the confusion. I wasn't able to synch up with Aliya on this issue before she went to Bangladesh and I didn't realize that she had already given verbal sign-off on the feature already.
Comments inline > I think that those points could be considered among feature > enhancements and could be scheduled in future releases. Exactly-- these can be future enhancements. No need for Nazir to redo any work. I do have a couple of quick questions: (*) Can we refer to the feature as "Account Documents" in the UI? Administrative documents isn't very intuitive. Seems like this would be a trivial change? (*) Since the functionality to upload and view an admin document is almost identical to the upload "report" template-- I hope that Nazir is planning to reuse the code that has already been built-- and that he's not building a separate code path-- since that will increase maintainability issues. Can you confirm. > Emily wrote: > >> Question for you and the list: Should we have error checking that > prevents a user from assigning a report to an "account document" > category if the report takes more than the account ID as a parameter? > Or if a report takes multiple parameters, should the report > link take the user to the parameter page in the Reports tab > to enter the additional information > >> > > > If I have rightly understood this issue, I think that an > error should prevent the user from adding a report that takes > more than the account ID as parameter when defining an > administrative document. Exactly-- because the report won't work if more parameters are required (since there is no parameters UI when selecting a admin report). > Would you please note that Nazir is constrained by a delay to > release this feature. Can you explain why Nazir is constrained by a delay? Regardless, I don't think it's a critical feature to add into this release. We just need to make sure the application handles this error gracefully-- ie, there's not a http 500 error or another server error if a more complicated report is uploaded. Can you confirm? Thanks Nesrine and again I apologize for the confusion. At least we have some good enhancement ideas for the next release. :-) Emily. ------------------------------------------------------------------------- SF.Net email is sponsored by: Check out the new SourceForge.net Marketplace. It's the best place to buy or sell services for just about anything Open Source. http://sourceforge.net/services/buy/index.php _______________________________________________ Mifos-functional mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/mifos-functional
