Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes #7730,BZ1127090 - Activation key max host count can be set from finite n... #4682

Merged
merged 1 commit into from Sep 30, 2014

Conversation

adamruzicka
Copy link
Member

...umber to infinite

@ehelms
Copy link
Member

ehelms commented Sep 26, 2014

@adamruzicka is there a Redmine issue for this bug?

@adamruzicka
Copy link
Member Author

@ehelms couldn't find any relevant on Redmind, created new one for it http://projects.theforeman.org/issues/7730

@ehelms
Copy link
Member

ehelms commented Sep 29, 2014

@adamruzicka thanks! could you also update your commit message to Fixes #7730 - ...

@cfouant
Copy link
Member

cfouant commented Sep 29, 2014

ACK

@adamruzicka adamruzicka changed the title Fixes BZ1127090 - Activation key max host count can be set from finite n... Fixes #7730,BZ1127090 - Activation key max host count can be set from finite n... Sep 30, 2014
@adamruzicka
Copy link
Member Author

@ehelms sorry, forgot to rename it after creating the redmine

@ehelms
Copy link
Member

ehelms commented Sep 30, 2014

@adamruzicka one last change, for our system to work properly we need the commit message (and not the pull request message) to reflect what is being fixed. If you wouldn't mind amending the commit message with the same thing the new PR message says that would be great.

@adamruzicka
Copy link
Member Author

@ehelms sure thing, done

@ehelms
Copy link
Member

ehelms commented Sep 30, 2014

@adamruzicka thanks!

ehelms added a commit that referenced this pull request Sep 30, 2014
Fixes #7730,BZ1127090 - Activation key max host count can be set from finite n...
@ehelms ehelms merged commit cd779b7 into Katello:master Sep 30, 2014
@adamruzicka adamruzicka deleted the bz1127090-activation_key branch October 1, 2014 07:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants