Conversation
mogoodrich
left a comment
There was a problem hiding this comment.
Thanks @Dutervil and sorry for the delay in reviewing!
This generally looks good, my main comment is since this just appears to be a copy of the StatusFragmentController with some new functionality, could we just expand the StatusFragmentController to support date parsing and the other formatting support we made here, or do we need this new class?
I'm happy to keep this as-is if need be, but I'd also supportive of adding new functionality to statusData.gsp if that makes things easier.
mseaton
left a comment
There was a problem hiding this comment.
@Dutervil - I tend to agree with @mogoodrich . Can you explain the purpose of this new controller and file? Why are we adding this in?
|
FYI @mseaton , @mogoodrich, @Dutervil |
and VisitStatusFragmentController.java
Creating controller and .gsp file for Prenatal Visit Status @mogoodrich @louidorjp