Thanks to visit codestin.com
Credit goes to github.com

Skip to content

feat(api,ui): add permission on node and delete permissions on env, app and pipelines#3902

Merged
yesnault merged 48 commits intomasterfrom
delete_perm
Feb 11, 2019
Merged

feat(api,ui): add permission on node and delete permissions on env, app and pipelines#3902
yesnault merged 48 commits intomasterfrom
delete_perm

Conversation

@bnjjj
Copy link
Contributor

@bnjjj bnjjj commented Feb 1, 2019

Benjamin Coenen added 24 commits January 21, 2019 11:35
wip
Signed-off-by: Benjamin Coenen <[email protected]>
wip
Signed-off-by: Benjamin Coenen <[email protected]>
Signed-off-by: Benjamin Coenen <[email protected]>
Signed-off-by: Benjamin Coenen <[email protected]>
Signed-off-by: Benjamin Coenen <[email protected]>
Signed-off-by: Benjamin Coenen <[email protected]>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

loadWorkflowNodeGroups is unused (from deadcode)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Error return value of migrateApplication is not checked (from errcheck)

@yesnault yesnault merged commit f7ab12e into master Feb 11, 2019
@yesnault yesnault deleted the delete_perm branch April 30, 2019 11:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

7 participants

Comments