Conversation
|
@MaximeBICMTL only requesting another review, but we already talked about the JS/TS update to be done later on. |
|
@regisoc Just saw your review request, will look into that PR and test it whenever I have the time (will be done this month). |
MaximeBICMTL
left a comment
There was a problem hiding this comment.
I found some bugs during testing (during the dictionary import part in particular), see PR regisoc#1 for the bug fixes that made it work for me.
Once the fixes are in I'll approve this PR.
REDCap UI bug fixes from testing
|
@MaximeBICMTL last quick review, and it can go to Dave as a last check. |
|
@MaximeBICMTL Sure why not, but I don't know much about React and Typescript, and sending something that I am not able to at least reasonably understand is a no-no. It was already discussed previously, and pretty sure anyone with React and Typescript can tackle that. |
|
@driusan ready for final review. |
Brief summary of changes
This PR adds a UI for designated users with new permission
redcap_ui_view.The UI is mainly targeted at people managing REDCap data integration, and adapts to one or multiple connected REDCap instances and projects.
The current UI composed by 3 tabs (more can be added) to give information and help navigate, this is particularly useful for issues arising when importing REDCap data e.g. missing notifications, network issues, out-of-sync instruments dictionary.
Images of current included tabs
Notifications received:
REDCap Dictionary in use:
REDCap import issues in issue tracker:
This PR also updates the
redcap2linst:module/redcap/toolsas a module specific tool.redcap_dictionarytable update when parsing the redcap dictionary.To discuss:
Testing instructions (if applicable)
redcapmodule in modules manager.Tools > REDCapmenu item.