Skip to content
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

Add to configuration server_processes_manager module #9189

Merged
merged 5 commits into from
Apr 18, 2024

Conversation

GeorgeMurad
Copy link
Contributor

@GeorgeMurad GeorgeMurad commented Apr 2, 2024

Testing instructions (if applicable)

  1. Go to the server processes manager without setting up LORIS-MRI code path /opt/Loris-MRI/bin/mri/ in the configuration module.
  2. See that page doesn't load.
  3. See that this is not mentioned in the README or TestPlan.

Link(s) to related issue(s):

@GeorgeMurad GeorgeMurad self-assigned this Apr 2, 2024
Removing the path
Re-type the Note
Removing the actual path
Adjust the configuration
@CamilleBeau CamilleBeau added the Passed manual tests PR has been successfully tested by at least one peer label Apr 18, 2024
@driusan driusan merged commit fe303a4 into aces:main Apr 18, 2024
28 checks passed
@ridz1208 ridz1208 added this to the 26.0.0 milestone Jun 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Passed manual tests PR has been successfully tested by at least one peer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants