-
Notifications
You must be signed in to change notification settings - Fork 27
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
simple tree: bare-bones tree view for documentation purposes #1139
Conversation
* A new bare-bones live tree view not available in production mode. * Indended to serve as documentation and a guide for future view development.
74a643d
to
1cb0769
Compare
src/views/Workflow.vue
Outdated
if (this.environment !== 'PRODUCTION') { | ||
// dynamically load development views that we don't want in production | ||
import('@/views/SimpleTree').then((SimpleTreeView) => { | ||
this.views.push(SimpleTreeView.default) | ||
}) | ||
} |
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.
Reviewers: Please check that the SimpleTree is present when the UI is built in dev mode (i.e. build:watch
) but not when run in production mode (i.e. build
).
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.
Checked, all good.
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.
me too
Rebased and ready. |
No tests for this ATM (functional documentation). Wouldn't object if anyone wanted to write one though... |
Resolved Conflicts: src/store/workflows.module.js src/views/Workspace.vue
@oliver-sanders - I've pushed a small conflict resolution commit (merge from master) to your branch. |
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.
Excellent. Nicely documented, works as intended.
@MetRonnie or @wxtim - I'm sure this would be a quick review for one of you. |
The only thing I might suggest is excluding the new file from code coverage? |
SimpleTree: a simple view a functional documentation
development.
Requirements check-list
CONTRIBUTING.md
and added my name as a Code Contributor.