-
Notifications
You must be signed in to change notification settings - Fork 73
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
Fixes #10157: Add an option to not display rule status/recent changes in directives screen #1482
Conversation
So, it seems that github don't want to let me edit the head description. So it is WIP, no real test done for now. |
PR rebased |
3418432
to
31d92bb
Compare
PR rebased |
31d92bb
to
a811d67
Compare
PR rebased |
a811d67
to
d063aca
Compare
PR rebased |
d063aca
to
a64a009
Compare
, showRulePopup : (Option[Rule]) => JsCmd | ||
, showRulePopup : (Option[Rule]) => JsCmd | ||
, columnCompliance : DisplayColumn | ||
, graphRecentChanges : DisplayColumn |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
':' Not correctly aligned :)
@@ -152,6 +154,8 @@ function generateRecentGraph(id, displayGraph) { | |||
|
|||
function recentChangesText(id) { | |||
|
|||
console.log("Writting as text " + id); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
console.log
@@ -116,6 +116,8 @@ function computeChangeGraph(changes, id, currentRowsIds, changeCount, displayGra | |||
|
|||
function generateRecentGraph(id, displayGraph) { | |||
if (displayGraph) { | |||
console.log("Writting as graph " + id); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
console.log
Apart the litlle remarks it's ok to me! |
… in directives screen
PR rebased |
a64a009
to
0dbb536
Compare
This PR is not mergeable to upper versions. |
OK, merging this PR |
OK, merging this PR |
2 similar comments
OK, merging this PR |
OK, merging this PR |
https://www.rudder-project.org/redmine/issues/10157