Log view mutation #1162
Closed
Log view mutation #1162
Conversation
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
- Allow a mutation argument type to be associated with a cylc object type (cycle, task etc.) without auto filling that object into the form input - Add icon for Edit Runtime
Uses introspection to expand fields of the query if none specified
Update introspection query response in mock service
Avoids need to manually reset state of form
Because that's the order they appear in the broadcast
Simplify help icon logic - it now shows help on hover instead of click
Adapt AOTF's handling of 'INPUT_OBJECT' for 'OBJECT' seeing as the former is not used?
- Remain open with loading spinner in the submit button while awaiting the mutation response - Show error snackbar and stay open on failed submission - Show warning snackbar and stay open if the Edit-Runtime form does not have any changes on submission
* Each node in the store is now associated with its UID type (user, workflow, cycle, task, job, $edge, $namespace). * Use this type for mutation listing rather than trying to infer whether a task is a family or not by looking at its node contents. * Simplify some internals.
* If a view's query changes (e.g. if its variables change) whilst it is running. The view will now be un-subscribed, then re-subscribed with the new query.
* Adds an interface for opening views via the eventBus. * Adds an `initialData` prop for views to recieve arguments.
|
@oliver-sanders - did you intend not to milestone this? |
|
Yes, some of the commits from here are cherry-picked onto Mel's log branch, I'll take whatever remains and put this up for review if needed. |
|
This will hopefully be superseeded shortly as part of a fix, will leave this open for a bit as a placeholder. |
|
Superseded by #1158? |
|
Yes, this also had the subscription update code which is now superseeded by #1249 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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 are a couple of things the log view will need which we hadn't implemented interfaces for yet:
Requirements check-list
CONTRIBUTING.mdand added my name as a Code Contributor.