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

app: Redirect to cockpit #1649

Merged
merged 1 commit into from Jul 10, 2017
Merged

app: Redirect to cockpit #1649

merged 1 commit into from Jul 10, 2017

Conversation

petervo
Copy link
Contributor

@petervo petervo commented Jul 5, 2017

Redirects the user to cockpit including a valid API token as a url fragment so that cockpit can validate the
current user.

Split from ManageIQ/manageiq#12506 post UI refactoring.

@petervo petervo force-pushed the cockpit-ws branch 2 times, most recently from a59f7da to 03422f2 Compare July 5, 2017 09:24
@petervo
Copy link
Contributor Author

petervo commented Jul 5, 2017

@mzazrivec so making a new PR seems to have helped. The current failures seem to be present in other PRs as well. Is this a just keep trying situation? Or is there a known good commit I should base off of?

Redirects the user to cockpit including a valid API
token as a url fragment so that cockpit can validate the
current user.
@miq-bot
Copy link
Member

miq-bot commented Jul 5, 2017

Checked commit petervo@ff78ac6 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
2 files checked, 0 offenses detected
Everything looks fine. 👍

@dclarizio
Copy link

@petervo we are having some travis failures right now, but are working to correct them. Basing on a different commit won't help as travis does a dynamic rebase off master before running the tests. Once the tests are fixed, we can restart your tests.

@mzazrivec mzazrivec added this to the Sprint 64 Ending Jul 10, 2017 milestone Jul 10, 2017
@mzazrivec mzazrivec merged commit ce0b54c into ManageIQ:master Jul 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants