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

fix for 6238 "Unable to view certificates issued by Sub CA in Web UI" separated from pr31 #63

Closed
wants to merge 2 commits into from

Conversation

pvoborni
Copy link
Member

@pvoborni pvoborni commented Sep 6, 2016

Pavel's patch separated from pr #31. Pavel is on vacation so he cannot split it.

Pavel Vomacka added 2 commits September 6, 2016 22:04
Sometimes the entity_show command must be called with options which are gathered
from result of entity_find command. These options needs to be passed as
arguments in URL which points to details page.

This functionality is implemented to table facet. There is new property
'additional_navigation_arguments' which is prepared for array of attributes
which will be passed to URL.

Part of: https://fedorahosted.org/freeipa/ticket/6238
The cert-show command needs to be called with cacn option. Cacn option is
passed using URL attribute.

https://fedorahosted.org/freeipa/ticket/6238
@pvoborni pvoborni added the ack Pull Request approved, can be merged label Sep 6, 2016
@pvoborni
Copy link
Member Author

pvoborni commented Sep 6, 2016

These 2 commits were already ACKed in PR #31 so we can merge them.

@martbab martbab added the pushed Pull Request has already been pushed label Sep 7, 2016
@martbab martbab closed this Sep 7, 2016
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
2 participants