Skip to content

Feature/multiple build plans#30

Merged
saatkamp merged 7 commits intomasterfrom
feature/multipleBuildPlans
Jun 26, 2019
Merged

Feature/multiple build plans#30
saatkamp merged 7 commits intomasterfrom
feature/multipleBuildPlans

Conversation

@lharzenetter
Copy link
Copy Markdown
Member

Short Description

To support the execution of a specific termination or build plan, the user can now select the interface and operation he wants to execute.

image

Proposed Changes

  • select an interface to be executed instead of a plan

revert operations interface since it would result in more changes in the container
@lharzenetter lharzenetter requested a review from fakepk March 25, 2019 10:17
Comment thread src/app/application-management/application-detail/application-detail.component.ts Outdated
};
// TODO we have to create a proper model for interface ressources delivered by the backend
return this.http.get<{ interfaces: Array<any> }>(serviceTemplateInstance._links['boundarydefinitions/interfaces'].href, httpOptions)
return this.http.get<InterfaceList>(serviceTemplateInstance._links['boundarydefinitions/interfaces'].href, httpOptions)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Does this fix the todo in line 78? If so, delete todo.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Unfortunately not. The container still creates a map of operations instead of a list.
The comment refers to lines 83-94.

Comment thread src/app/application-management/application-management.reducer.ts
@lharzenetter lharzenetter force-pushed the feature/multipleBuildPlans branch from 18a051a to 9087a91 Compare June 26, 2019 08:22
@saatkamp saatkamp merged commit 0eebe9c into master Jun 26, 2019
@saatkamp saatkamp deleted the feature/multipleBuildPlans branch June 26, 2019 08:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants