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

Added a fix for setting Priority as required field #83

Closed
wants to merge 1 commit into from

Conversation

Akasurde
Copy link
Member

@Akasurde Akasurde commented Sep 15, 2016

Fixes: https://fedorahosted.org/freeipa/ticket/6335

Signed-off-by: Abhijeet Kasurde akasurde@redhat.com

});
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please, turn off newlines at the end of file in your text editor and/or fix this. Other than that LGTM.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK

@stlaz
Copy link
Contributor

stlaz commented Sep 15, 2016

Looks OK to me, thanks.

@stlaz stlaz added the ack Pull Request approved, can be merged label Sep 15, 2016
@Akasurde
Copy link
Member Author

@stlaz Thanks

@MartinBasti
Copy link
Contributor

Sorry guys, but ticket is in closed milestone you need a new one.

@MartinBasti MartinBasti removed the ack Pull Request approved, can be merged label Sep 15, 2016
@Akasurde
Copy link
Member Author

@mbasti-rh I re-opened the ticket, Could you please provide ack ?

@MartinBasti
Copy link
Contributor

@Akasurde I said we need a new ticket for it. I opened one for you https://fedorahosted.org/freeipa/ticket/6335

… Details facet

Fixes: https://fedorahosted.org/freeipa/ticket/6335

Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
@stlaz stlaz added the ack Pull Request approved, can be merged label Sep 16, 2016
@MartinBasti MartinBasti added the pushed Pull Request has already been pushed label Sep 21, 2016
@MartinBasti
Copy link
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged pushed Pull Request has already been pushed
Projects
None yet
3 participants