-
Notifications
You must be signed in to change notification settings - Fork 2
Development at PNL #3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
tashrifbillah
wants to merge
612
commits into
dptools:master
Choose a base branch
from
AMP-SCZ:pnl-devel
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The websockets don't work so let's remove them
* Installed React hook form * Created New Config Page to handle form
* Refactored Edit config page to use latest updates
* Added Share Form to config page * Updated components to be reused * updated props * Updated imports to use components file * Fixed tests * Moved Config Page configlist logic to parent component
Tashrif removed them in a562f1e but it is actually needed by the celery worker defined in supervisord.conf
Revert mongo to v4.4
see issue #337
* Add react hook form
* This pr moves the javascript from the app_build directory to the public directory
* This improves the backend handling of th matrix data * It uses a service and a data processor * Study permissions are changed * Processing time for data has been reduced
* Move build process into image
* Removed unused css
* Added docker compose file
* Provision the Node server dockerfile * Updated code to load env variables instead of config * Updated docker compose to use dockerfile
362 add rabbitmq to docker
355 consolidate mongo dbs
This reverts commit 7a8076a.
…llection" This reverts commit d1b60d2.
…-dbs Revert "355 consolidate mongo dbs"
* Fix crash from tooltip * added fix to not let tooltip crash * PR comments * fix from comments
* PNG download * fix png download * Added condition * Added condition * Renamed variable * renamed var
* UI to save as draft * Added workflows tests and ui designs to implement saving a configuration as a draft * lint fix * add lint fix
* Added recruitment status filters with tests
* Implement new configurations form stylings designs and directives
* bugfix Matrix configurations
* Additional consoles * consoles * Send error object * Add error object to response
* Changed insertOne to findAndModify * On documentDb it looks like insertOne is no longer supported? * Updated to findAndModify with upsert as true * dd error * add error to delete * Funnel errors to logger * Funnel errors to logger * add back consent * unsure about null consent * fx test * Fix test * additional logs * Added additional logs
* Clicking learn more button navigates to dpdash page
* Change update to upsert * in the logs it looks like the update operation could also not work * remove update * removed updateone method
* npm fix ip dep
* We are getting incorrect counts and it's because the assessment name is different
* Provides easy to configure SSL proxy * Add a mock smtp server for local development
* clarify nginx-proxy-manager launch * clarify risk with self-signed certificates
Patch on prem deployment instruction
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR is essentially a copy of PR #2. The latter one got closed inadvertently when we renamed our branch. Sorry about the inconvenience.
Please see the review comments in PR #2.