-
Notifications
You must be signed in to change notification settings - Fork 43
Conversation
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.
Something went wrong with the rebase/merge of your branch towards the dev
branch. The availability to define labels to resources would be removed by your commit.
This needs to be corrected before merging the pull request
@@ -236,11 +240,6 @@ | |||
"description": "name of the service instance", | |||
"title": "name of the service instance" | |||
}, | |||
"labels":{ |
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.
The functionality of labels are removed with your commit - there seems to be something wrong with the rebase/merge
libs/btpsa-usecase.json
Outdated
@@ -226,11 +226,6 @@ | |||
"description": "name of the service instance", | |||
"title": "name of the service instance" | |||
}, | |||
"labels":{ |
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.
The functionality of labels are removed with your commit - there seems to be something wrong with the rebase/merge
Great catch! Don't know what happened. I've just fixed the issue and it should be fine now. Would you mind checking again? |
This PR addresses the issue #313 so that required parameters for subscribing to apps are provided in the btpsa-usecase.json library.