Issue510notpatientindb

Merged
#96 · Created  · Last updated

Merged pull request

Merged in issue510notpatientindb (pull request #96)

b7f39f7·Author: ·Closed by: ·2017-07-12

Description

Thanks for your help with reviewing this for me chaps. One last thing to look at - http://testing.openrem.org/openrem/ has the latest version of this and I have added a question to the top of homepage if you are logged in as an admin. If you clear it, the only way to get it back currently is to use the Django admin interface.

  • Very basic initial view of not patient indicators. Refs #510

  • Completed the view of indicators page, linked to django admin interface to edit them. Removed all other models from admin interface for now - need to make sure this is ok. Haven't linked db fields into import checks yet. Refs #510

  • Committed and pushed to stage without a link in admin menu! Try again! Refs #510

  • Improved grammar, added em tags. Refs #510, will stage again.

  • First attempt to implement using patterns from the database with fnmatch(). Refs #510

  • Corrected and improved get_not_pt and added a test suite Refs #510

  • Doc strings for tests. Refs #510

  • Added in 0.7.4 no-patient patterns to Siemens CT import tests to enable testing of ref #510 with real data.

  • Overhaul of not-patient page. Added function to setup matching patterns to continue 0.7.4 behaviour. Refs #510

  • Modified the tests to reflect the change to record patterns in lower case. Refs #510

  • ID code example had gone missing! Refs #510

  • Changing the layout and content of not-patient template. Refs #510

  • Changed colours, minor text, put example and wildcards into two columns. Refs #510

  • Took example out of panel, other minor mods. Refs #510

  • Resized to fit md-8. Refs #510

  • Resized to fit md-10. Refs #510

  • Created the start of the docs for non-patient indicators. Refs #510

  • A little more on the docs. Added link from interface. Refs #510

  • Attempting a more simple table. refs #510

  • Attempt to get start to remain as star. Refs #510

  • Added explanation of the patterns. Refs #510

  • Minor change to example. Refs #510

  • Given in - will be doing forms for not-patient indicators after all. Refs #510

  • Added add, modify and delete links for names. Refs #510

  • Full set of views and templates with basic text for adding, modifying, and deleting name patterns. Needs text review. Refs #510

  • Added the current docs text to the add/change form template. Refs #510

  • Added link to replicate 0.7.4. Refs #510

  • Moved most of the create/modify template to share it between name and ID. Refs #510

  • Converted ID patterns to be managed in the same way as name patterns. Refs #510

  • Adding a note about not affecting existing imports. Refs #510

  • Changed text in bottom panel. Refs #510

  • Removed a p

  • It appears that this file was never committed? Refs #510

  • Added new config menu image, changed existing config menu topics. trying to have just one. Refs #510

  • Added note about pre-0.8 and config menu. Refs #510

  • Trying to make width of image appropriate. Refs #510

  • Moving the menu up the page and making it slightly bigger. Refs #510

  • Reducing the size of the config menu on delete settings, seeing if next image will slide alongside nicely. Refs #510

  • Alignment was wrong for second image.

  • Added a clearfix that probably won't work. Added brief instructions on how to add/modify/delete patterns. Changed the case of Name. Refs #510

  • Removed alternative admin menus, hopefully all redundant now. Formatted other uses of new config menu. Refs #510

  • More formatting. Refs #510. Also removed the section about finding your IP address and changed it for using 0.0.0.0

  • Now tests for admin group in more places and removes links if not admin. Adds messages to confirm success and error message if user still manages to craft correct URL. Refs #510, credit to @tcdewit for reminding me to check!

  • Not really related to ref #510, but it has been bothering me. @dplatten - any objection to removing chart message when not logged in?

  • Added a panel to the home page and a singleton model for recording if not-patient indicator question should be asked. Refs #510

  • Now passing admin questions as a new dict plus a bool to determine whether to display the panel or not. Refs #510

  • Fixed test for admingroup so it works when not in that group! Refs #510

  • Mainly file encoding ref #503

  • Prompt to fix not-patient identifiers on home page now working. Not particularly generic, but could be extended for other questions. Refs #510

  • Added codacy related coverage xml file to gitignore.

0 attachments

0 comments

Loading commits...