Skip to content

component to run xlr given a template id and task name.#122

Merged
shakeh merged 2 commits intorelease/3.5.2from
feature/xlr
Apr 5, 2019
Merged

component to run xlr given a template id and task name.#122
shakeh merged 2 commits intorelease/3.5.2from
feature/xlr

Conversation

@shakeh
Copy link
Collaborator

@shakeh shakeh commented Mar 28, 2019

No description provided.

@shakeh shakeh changed the base branch from develop to release/3.5.2 March 28, 2019 23:37
@shakeh shakeh requested a review from dlamoris March 28, 2019 23:38
@@ -0,0 +1,92 @@
'use strict';

angular.module('mms.directives')
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

module should be mmsApp

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

forgot to update when i moved it to a ve level directive.

config.mathJaxLib = 'https://cdnjs.cloudflare.com/ajax/libs/mathjax/2.7.1/MathJax.js?config=TeX-AMS-MML_HTMLorMML';
config.allowedContent = true;
config.allowedContent = true;
config.extraAllowedContent = 'mms-run-xlr';
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not needed, since allowedContent is already true

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hmm i'll look at it again... i think the editor was clearing the custom tag in source

@shakeh shakeh requested a review from dlamoris April 3, 2019 05:34
@shakeh shakeh merged commit e86fed9 into release/3.5.2 Apr 5, 2019
@dlamoris dlamoris deleted the feature/xlr branch January 16, 2020 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants