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

[Core/Dicom_Archive] Markdown File Help Content #2196

Merged
merged 6 commits into from
Sep 28, 2016

Conversation

driusan
Copy link
Collaborator

@driusan driusan commented Sep 19, 2016

This adds the help content from markdown format files in the module help/ directory.

This should be significantly easier to maintain and review than storing the help content in the database where everything requires an SQL patch to update. It's also faster.

This is only available for "new" style modules, and the dicom_archive (which previously had no help at all) is used as an example.

Only a subset of markdown is supported. Namely:

  • Paragraphs (by separating text with an empty line)
  • Headers (by prefixing a paragraph with up to six # characters. Note that unlike real markdown, there must be an empty line before the first paragraph of the section)
  • Bold and italics with * or **
  • Markdown style links

(The new React "Markdown" component can be used to render markdown other places in LORIS, if required.)

@driusan driusan added Feature PR or issue introducing/requiring at least one new feature Discussion Required PR or issue awaiting the resolution of a discussion between all involved parties labels Sep 19, 2016
@driusan driusan added this to the 17.0 milestone Sep 19, 2016
@codecov-io
Copy link

codecov-io commented Sep 20, 2016

Current coverage is 13.76% (diff: 9.09%)

Merging #2196 into 17.0-dev will decrease coverage by <.01%

@@           17.0-dev      #2196   diff @@
==========================================
  Files           121        121          
  Lines         20241      20249     +8   
  Methods        1132       1133     +1   
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits           2787       2788     +1   
- Misses        17454      17461     +7   
  Partials          0          0          

Powered by Codecov. Last update 096d08a...ef91c98

@christinerogers christinerogers added the Documentation PR or issue introducing/requiring modifications to the code documentation (test plans, wikis, docs) label Sep 20, 2016
@driusan driusan added Request Code Review Add to Release Notes PR change should be highlighted in Release notes (important security, features and bugfixes) and removed Discussion Required PR or issue awaiting the resolution of a discussion between all involved parties labels Sep 21, 2016
@driusan driusan added the Needs Rebase PR contains conflicts with the current target branch or was issued to the wrong branch label Sep 23, 2016
Dave MacFarlane added 6 commits September 28, 2016 09:28
This should be significantly easier to maintain and review than storing
the help content in the database where everything requires an SQL patch
to update. It's also significantly faster.

This is only available for "new" style modules, and the dicom_archive
(which previously had no help at all) is used as an example.
@driusan driusan removed Needs Rebase PR contains conflicts with the current target branch or was issued to the wrong branch Request Code Review labels Sep 28, 2016
@driusan driusan merged commit 4b94cee into aces:17.0-dev Sep 28, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Add to Release Notes PR change should be highlighted in Release notes (important security, features and bugfixes) Documentation PR or issue introducing/requiring modifications to the code documentation (test plans, wikis, docs) Feature PR or issue introducing/requiring at least one new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants