-
Notifications
You must be signed in to change notification settings - Fork 256
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
TICKscript Editor #1721
Merged
+1,737
−130
Merged
TICKscript Editor #1721
Changes from all commits
Commits
Show all changes
44 commits
Select commit
Hold shift + click to select a range
423f02b
Add skeleton for tickscript editor page
121watts b5aa82f
Introduce TICKscript editor
121watts 202702b
Introduce CodeMirror for TICKscript editor
121watts ad1f78a
Polish the editor
121watts 8722120
Display TICKscript on rules index
121watts 1e1c955
Polish alert rules table
alexpaxton c4562aa
Update CHANGELOG
121watts b66be55
Update kapacitor links to include tasks and ping paths
121watts b6880c2
ES6ify
121watts 4f1372a
Introduce skeleton to create task through proxy
121watts b6ca021
WIP Edit scripts returned from the server
121watts cea9d07
Introduce TICKscript editing
121watts 9f371f1
Set default props for ruleID
121watts 0e8fecf
Add lambda to syntax highlighting
121watts 4437475
Apply selected dbrps to state
121watts fa2192f
Prettier
121watts 026b1f1
Add comment
121watts 018bd17
Add ability to apply changes per click
121watts 1cdcc80
Make UserRow work with new MultiSelectDropdown shape
121watts 74145b0
Cleanup checks with default props
121watts 03e40c8
Cleanup
121watts f4aeefa
Add tickscript type selector
121watts a1ecdb6
WIP Introduce tickscript id editing
121watts 879685d
Remove view only Tickscript
121watts fe6c5ac
Cleanup
121watts c653982
Make task ids only edit when making new task
121watts 25d5310
Prettier
121watts 09d554c
Organize styles and use custom theme for code mirror
alexpaxton 077bdf2
Change layout of tickscript editor
alexpaxton cce44b1
Update to react-codemirror 1.0
121watts b817c82
Update branch with new linter rules
121watts b151e81
Use /rules to create tickscripts
121watts 4915e97
Update existing tickscript
121watts f7d59f6
Fix auto indendation to match indentation of previous line
121watts 701c4bf
Publish notification after route change
121watts dc7fc85
Remove TICKscript name creation
121watts 6dd8e37
Use tickscript id as 'name' for raw tickscripts
121watts e646f42
Add a TICKscript table to alert rules page
121watts e314789
Refine copy
121watts 2fdee35
Update CHANGELOG
121watts 9059da0
Merge branch 'master' into feature/tickscript-editor
121watts f5209a8
Fix multiselect in Admin page
121watts 4abe50e
Merge branch 'feature/tickscript-editor' of github.com:influxdata/chr…
121watts abbcd6c
Remove react-addons-update
121watts File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
Oops, something went wrong.
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.
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.
There's something wrong with the permissions dropdowns on Enterprise, they list items but don't display the text.
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.
Also, selecting one selects them all.
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.
This has been fixed, but other issues with the backend have come up. See #2021