Skip to content

Conversation

@behrens-web
Copy link

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.

@hrj
Copy link
Member

hrj commented May 13, 2021

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants