mdsupport wrote on Wednesday, April 10, 2013:
This enhancement to the notes makes patient issues information available in addition to the basic patient data.
bradymiller wrote on Saturday, April 13, 2013:
Hi MD Support,
I placed a review on your github commit:
Just some minor things. When they are addressed. just let me know and then I’ll test it and then commit it to the official codebase if it tests ok.
thanks,
-brady
OpenEMR
mdsupport wrote on Saturday, April 13, 2013:
Done.
mdsupport wrote on Sunday, April 14, 2013:
Another fix - This one to stop NationNotes textarea displaying raw html code.
Since these notes are designed to be large blocks, showing small textbox with scroll lines does not help the users. It may be better to :
- display the full text upon mouseover like visit history
- save space by showing first 1 or 2 lines followed by a ‘change’ hyperlink.
bradymiller wrote on Sunday, April 14, 2013:
Hi MD Support,
I just committed your first fix/improvement to nation notes. very cool. thanks for the contribution.
The second fix I am a bit unclear of. Won’t this then open it up to cross-scripting attacks? I’ll play around with it a bit.
-brady
OpenEMR
mdsupport wrote on Sunday, April 14, 2013:
Second fix is a cosmetic band-aid to the peculiar approach chosen by the developer. No idea why they are using text area. In any case this content is already stored in the table and is decoded for the form display. In addition the edit process somehow decodes the field content as well. Could use that code if it has been vetted for cross-scripting.
vambati wrote on Tuesday, April 16, 2013:
can we make vitals available in nation notes?