Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 388219 - [Review] I'm asked many times for the remote name
Summary: [Review] I'm asked many times for the remote name
Status: RESOLVED FIXED
Alias: None
Product: Orion
Classification: ECD
Component: Git (show other bugs)
Version: 1.0   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 1.0 M2   Edit
Assignee: Maciej Bendkowski CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-08-28 09:09 EDT by Szymon Brandys CLA
Modified: 2012-08-30 11:38 EDT (History)
2 users (show)

See Also:
maciej.bendkowski: review? (Szymon.Brandys)


Attachments
Illustration (24.31 KB, image/png)
2012-08-28 09:13 EDT, Szymon Brandys CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Szymon Brandys CLA 2012-08-28 09:09:23 EDT
See the screenshot.
Comment 1 Szymon Brandys CLA 2012-08-28 09:13:24 EDT
Created attachment 220389 [details]
Illustration

The screenshot illustrates the state of my workspace when I started reviewing a contribution. In this particular example I'm trying to attach git@github.com:edytaprzymus/orion.client.git remote to my repo.
I named the new remote "edyta" clicked Ok and the prompt re-appeared. I named it again "edyta" and then I was asked to accept host and to give credentials.

Two things that should be fixed are:
- the prompt for the remote name should be a slideout
- I should be asked only once for the name

I'm adding Maciek to cc, since he already fixed similar issues on other Git pages.
Comment 2 Edyta Przymus CLA 2012-08-29 08:12:34 EDT
Maciek, could you please let me know if you plan to fix also this issue since you fixed some similar things already?
Comment 3 Maciej Bendkowski CLA 2012-08-29 08:21:29 EDT
(In reply to comment #2)
> Maciek, could you please let me know if you plan to fix also this issue
> since you fixed some similar things already?

Yes, I'm investigating it already.
Comment 5 Szymon Brandys CLA 2012-08-30 11:38:47 EDT
Fix merged to master. Thanks Maciek.