Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 446059

Summary:	Improvements to deployment UI
Product:	[ECD] Orion	Reporter:	Szymon Brandys <Szymon.Brandys>
Component:	Deployment	Assignee:	Malgorzata Janczarska <malgorzata.tomczyk>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	P3
Version:	unspecified
Target Milestone:	7.0
Hardware:	PC
OS:	Windows 7
Whiteboard:

Description Szymon Brandys

2014-10-06 10:29:50 EDT

1. In Manual Deployment Information when deploy is triggered and is finished with error, we should see the unknown state icon instead of forever spinning wheel. If one sets CheckState in error, then the state will be checked.

2. Getting launch configurations may take some time. When one opens the project's page, he could see the configurations after couple seconds. I would like to see the placeholder for configurations and any message showing that launch configurations are being computed.

3. method deploy() in cfDeployService gets a launch conf object with Params field, while getLaunchConfigurations gets objects with Parameters.

4. Be able to specify the log url in the plug-in instead of logLocationTemplate in service declaration

Comment 1 Malgorzata Janczarska

2014-10-06 11:06:19 EDT

4. Be able to specify the log url in the plug-in instead of logLocationTemplate in service declaration

This one is already done, getLogLocationTemplate(launchConf)

Comment 2 Malgorzata Janczarska

2014-10-09 14:32:04 EDT

(In reply to Szymon Brandys from comment #0)
> 1. In Manual Deployment Information when deploy is triggered and is finished
> with error, we should see the unknown state icon instead of forever spinning
> wheel. If one sets CheckState in error, then the state will be checked.

If we don't return any state I added checking for it. I think it's better then unknown state.

Comment 3 Malgorzata Janczarska

2014-10-10 08:44:22 EDT

(In reply to Szymon Brandys from comment #0)
> 3. method deploy() in cfDeployService gets a launch conf object with Params
> field, while getLaunchConfigurations gets objects with Parameters.

Fixed, now the deploy method takes the same params as ToSave.

Comment 4 Malgorzata Janczarska

2014-10-20 10:37:15 EDT

All points are done now.