gngr icon indicating copy to clipboard operation
gngr copied to clipboard

Proposed Solution to Issue 171

Open behrens-web opened this issue 4 years ago • 1 comments

The project I just submitted should display the number of accepted jobs and total jobs when the "Req Mgr" button is pressed on gngr. Accepted jobs will be on the left side of the "OK" button whereas Total jobs will be on the right side of the "OK" button.

I look forward to your input on my contribution.

behrens-web avatar May 12 '21 19:05 behrens-web

Hey @behrens-web

This is a great start.

Quick comment: can you avoid committing changes to files that are not relevant to the PR? I am not sure what were the changes (Github diff says "empty file") but they were probably some minor changes.

I assume you are not that familiar with git yet.

It will help if you can do something like this:

  • git diff master..HEAD > ~/gngrPatch.txt
  • This will save all your changes in a text file ~/gngrPatch.txt.
  • git checkout master
  • git checkout -b newBranch
  • git apply ~/gngrPatch.txt : This will reapply all your changes on the new branch
  • Then commit only the changes that are relevant to this feature, and ignore the rest.
  • Make a new PR

Once I can see only the relevant changes, I will also be able to compile and try the changes on my system.

best, HRJ

hrj avatar May 13 '21 19:05 hrj