-
Notifications
You must be signed in to change notification settings - Fork 76
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
Add check for default workflow #4118
Conversation
8c03e04
to
f59b0ce
Compare
PR UpdateWas able to produce the same error (can't read property |
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.
Can this be rebased with master? Other than that, it looks ready to go. Thanks!
4977764
to
2a07ab2
Compare
Rebased :) |
@jelliotartz Would you be up for writing unit tests for the project status page? Not a high priority, but another thing to do if you have time. Thanks! |
sure, will do! |
* Add check for default workflow. * Add project configuration check in handleToggle.
Describe your changes
default_workflow
is stored) on a project before conditionally rendering an asterisk next to the default workflow on the admin/project-status page.Review Checklist
rm -rf node_modules/ && npm install
and app works as expected?Optional
ChangeListener
orPromiseRenderer
components with code that updates component state?