Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 386845 - 'Pull Request' page should have better descriptions and header titles
Summary: 'Pull Request' page should have better descriptions and header titles
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: Edyta Przymus CLA
QA Contact:
URL:
Whiteboard: gsoc2012
Keywords:
Depends on:
Blocks: 386825
  Show dependency tree
 
Reported: 2012-08-08 10:56 EDT by Szymon Brandys CLA
Modified: 2012-08-22 06:19 EDT (History)
1 user (show)

See Also:


Attachments
new descriptions (107.41 KB, image/jpeg)
2012-08-13 04:45 EDT, Edyta Przymus CLA
no flags Details
Suggestion (89.55 KB, image/png)
2012-08-13 06:02 EDT, Szymon Brandys CLA
no flags Details
Suggestion 2 (78.79 KB, image/png)
2012-08-13 06:05 EDT, Szymon Brandys CLA
no flags Details
description2 (108.67 KB, image/jpeg)
2012-08-14 08:32 EDT, Edyta Przymus 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-08 10:56:18 EDT
I talked to Susan about that and it seems we can make better job on this page. For instance:
- if you have no repositories and you try to open commit to review, you should see:

You are trying to review contribution 160f4eefa006f6b74514a8f8a24a458a0b569e53 from ssh://git.eclipse.org/gitroot/orion/org.eclipse.orion.client.git
Unfortunately the commit can not be found in your workspace. To see it try one of the following:
1. Create new repository
[Clone Repository] using ssh://sbrandys@git.eclipse.org/gitroot/orion/org.eclipse.orion.client.git

- if you have the repository but the commit is not there, you should see:

You are trying to review contribution 160f4eefa006f6b74514a8f8a24a458a0b569e53 from ssh://git.eclipse.org/gitroot/orion/org.eclipse.orion.client.git
Unfortunately the commit can not be found in your workspace. To see it try one of the following:

1. Try to update your repositories
or
2. Attach the remote to one of existing repositories
or
3. Create new repository

- if the commit exists, we could have that:

You are trying to review contribution 160f4eefa006f6b74514a8f8a24a458a0b569e53 from ssh://git.eclipse.org/gitroot/orion/org.eclipse.orion.client.git

The commit can be found on the following repositories
....

Advanced v

you can also

1. Try to update your repositories
or
2. Attach the remote to one of existing repositories
or
3. Create new repository

I hope the idea is clear.
Comment 1 Edyta Przymus CLA 2012-08-09 18:18:29 EDT
https://github.com/edytaprzymus/orion.client/commit/12d2f369638f1a2e4b1429158861c548495106a6
My first solution for this problem
Comment 2 Edyta Przymus CLA 2012-08-12 13:20:01 EDT
Test
Comment 3 Edyta Przymus CLA 2012-08-12 18:26:53 EDT
https://github.com/edytaprzymus/orion.client/commit/e089bc401819e4bf942da035877df7ddcada2418

Commit better then the previous one.

I wrote all this code and have the rights to contribute it to Eclipse under the
eclipse.org web site terms of use.
Comment 4 Szymon Brandys CLA 2012-08-13 04:27:01 EDT
Can I see a screenshot?
Comment 5 Edyta Przymus CLA 2012-08-13 04:45:30 EDT
Created attachment 219796 [details]
new descriptions
Comment 6 Szymon Brandys CLA 2012-08-13 06:02:48 EDT
Created attachment 219798 [details]
Suggestion
Comment 7 Szymon Brandys CLA 2012-08-13 06:05:23 EDT
Created attachment 219799 [details]
Suggestion 2
Comment 8 Szymon Brandys CLA 2012-08-13 06:06:56 EDT
I would also start using wording suggested by Susan ie "Contribution Review", "Contribution Review Request" etc.
Comment 10 Szymon Brandys CLA 2012-08-14 06:24:13 EDT
Screenshots?
Comment 11 Edyta Przymus CLA 2012-08-14 08:32:38 EDT
Created attachment 219856 [details]
description2
Comment 12 Edyta Przymus CLA 2012-08-14 08:35:39 EDT
https://github.com/edytaprzymus/orion.client/commit/c1fa2080a001173de206c4d729b00c9503e23e80

Here is slightly better commit. BTW, in this commit I also fixed bug I observed about cloning function in Contribution Request page
Comment 13 Edyta Przymus CLA 2012-08-16 04:01:01 EDT
https://github.com/edytaprzymus/orion.client/commit/97c5cc33891f40a2a50811c1f31e5fa1e7a978f5
Please consider this commit, it is rebased to the master.
Comment 14 Szymon Brandys CLA 2012-08-17 08:49:50 EDT
Wording on pages looks fine, but you also need to change the page name, variables in the code, etc.
Comment 18 Edyta Przymus CLA 2012-08-17 17:31:44 EDT
https://github.com/edytaprzymus/orion.client/commit/7a75eaf67d89077f9c365680717beb9f7b2b1c30
Please consider this commit
Comment 21 Szymon Brandys CLA 2012-08-21 02:39:56 EDT
Still not good. The "Ask for Review" dialog shows http://localhost:8080/git/pullRequest.html... I guess the same is sent to reviewers.
Comment 23 Edyta Przymus CLA 2012-08-21 14:02:34 EDT
https://github.com/edytaprzymus/orion.client/commit/66059977ba20386fd3b3153028d9a7eead0a4579

I just found one more little trace of old name in client side, here it's removed
Comment 24 Szymon Brandys CLA 2012-08-22 06:19:45 EDT
Ok. Both client and server parts merged to master.