This Bugzilla instance is deprecated, and most Eclipse projects now use GitHub or Eclipse GitLab. Please see the deprecation plan for details.
Bug 385833 - Be able to add remote to existing repo on pullRequest page
Summary: Be able to add remote to existing repo on pullRequest page
Status: RESOLVED FIXED
Alias: None
Product: Orion (Archived)
Classification: ECD
Component: Git (show other bugs)
Version: 0.5   Edit
Hardware: PC All
: P3 normal (vote)
Target Milestone: 1.0 M1   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard: gsoc2012
Keywords:
Depends on: 383425
Blocks:
  Show dependency tree
 
Reported: 2012-07-24 06:10 EDT by Szymon Brandys CLA
Modified: 2012-08-08 10:25 EDT (History)
1 user (show)

See Also:


Attachments
Add remote dialog. You have to type remote name. (70.19 KB, image/jpeg)
2012-07-31 05:28 EDT, Edyta Przymus CLA
no flags Details
Something is broken in the latest version (22.80 KB, image/png)
2012-08-06 05:21 EDT, Szymon Brandys CLA
no flags Details
Wrong section title (15.46 KB, image/png)
2012-08-06 08:33 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-07-24 06:10:30 EDT
On pullRequest page we should be able to add remoteUrl to the existing clone. Right now we can just create a new clone based on the given url.
Comment 1 Edyta Przymus CLA 2012-07-26 13:49:45 EDT
https://github.com/edytaprzymus/orion.server/commit/ea5324333ab5a3a0f8e8b469621a24afb220774a
New section on PullRequest page, just at the bottom of it.
Comment 2 Szymon Brandys CLA 2012-07-31 05:21:38 EDT
Could you attach a screenshot, please?
Comment 3 Edyta Przymus CLA 2012-07-31 05:28:32 EDT
Created attachment 219354 [details]
Add remote dialog. You have to type remote name.
Comment 4 Edyta Przymus CLA 2012-07-31 08:27:41 EDT
I wrote all this code and have the rights to contribute it to Eclipse under the
eclipse.org web site terms of use.
Comment 5 Edyta Przymus CLA 2012-07-31 10:05:34 EDT
https://github.com/edytaprzymus/orion.client/commit/4046b2f3643376ca87a9da9468494ab64cece4a4

Fetching automatically after adding a remote is added.
Comment 7 Szymon Brandys CLA 2012-08-06 05:21:05 EDT
Created attachment 219577 [details]
Something is broken in the latest version

See areas marked with arrows.
Comment 8 Szymon Brandys CLA 2012-08-06 08:22:28 EDT
Use slideouts instead of prompts for collecting simple data like remote name. I tried to use "Add remote and fetch" for a remote that needs credentials. It failed.
Comment 9 Szymon Brandys CLA 2012-08-06 08:33:45 EDT
Created attachment 219588 [details]
Wrong section title

I started completely new user and have no repos. The header marked on the screenshot is obviously wrong.
Comment 10 Szymon Brandys CLA 2012-08-07 04:14:42 EDT
(In reply to comment #9)
> I started completely new user and have no repos. The header marked on the
> screenshot is obviously wrong.

I looked at the latest commit. The header is still wrong.

I used this url http://localhost:8080/git/pullRequest.html#git://github.com/edytaprzymus/orion.client.git_1d193f163aaa2e301699fd85c16cd2d1eccdcf95 on my local Orion server for a brand new user (no repos). I chose to clone, but when it was done I still saw no commit to open and no repos to fetch.

So it seems to be worse than the previous version.
Comment 11 Edyta Przymus CLA 2012-08-07 10:29:42 EDT
Szymon,

did yoy try this commit? https://github.com/edytaprzymus/orion.client/commit/c0ef89005f6e23ae41ec984214b7e9bd5d1aa16d 
Because I just made test case you described in latest comment and such a behaviour didn't occur..
Comment 12 Szymon Brandys CLA 2012-08-07 11:51:11 EDT
Please rebase the code and make sure it still works for you.
Comment 13 Edyta Przymus CLA 2012-08-07 12:10:46 EDT
https://github.com/edytaprzymus/orion.client/commit/ca5d6ee7adf64c81f40a842cc18fda4c218f94ac

Rebased commit - it works for me as supposed.
Comment 14 Szymon Brandys CLA 2012-08-08 06:35:08 EDT
(In reply to comment #13)
> https://github.com/edytaprzymus/orion.client/commit/ca5d6ee7adf64c81f40a842cc18fda4c218f94ac
> 
> Rebased commit - it works for me as supposed.

I guess "Choose repository" key in gitmessages.js is no more used.
Comment 15 Szymon Brandys CLA 2012-08-08 10:25:17 EDT
I fixed remaining issues and released the fix to master.