Merged in issue510notpatientindb (pull request #96)
b7f39f7·Author: Ed McDonagh·Closed by: Ed McDonagh·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
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
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.