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

WebUI: Fix showing vault in selfservice view #651

Closed
wants to merge 1 commit into from

Conversation

pvomacka
Copy link

@pvomacka pvomacka commented Mar 24, 2017

Vaults menu item was shown even when the KRA service was not installed.
That was caused by different path to the menu item in admin's view
and in selfservice view.

The path is now set correctly for both situations. 'network_service/vault'
for admin's view and 'vault' for selfservice view.

https://pagure.io/freeipa/issue/6812

Vaults menu item was shown even when the KRA service was not installed.
That was caused by different path to the menu item in admin's view
and in selfservice view.

The path is now set correctly for both situations. 'network_service/vault'
for admin's view and 'vault' for selfservice view.

https://pagure.io/freeipa/issue/6812
@pvoborni
Copy link
Member

Works fine. ACK

@pvoborni pvoborni added the ack Pull Request approved, can be merged label Mar 24, 2017
@pvomacka pvomacka added the pushed Pull Request has already been pushed label Mar 27, 2017
@pvomacka
Copy link
Author

ipa-4-5:

  • 7b3cb1c WebUI: Fix showing vault in selfservice view
    master:

  • ab6d7ac WebUI: Fix showing vault in selfservice view

@pvomacka pvomacka closed this Mar 27, 2017
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