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

BZ#1446405-Update skinning documentation #727

Merged
merged 1 commit into from Apr 29, 2017

Conversation

AllenBW
Copy link
Member

@AllenBW AllenBW commented Apr 28, 2017

Indicate how to change 404 and order details logo
@AllenBW AllenBW added this to the Sprint 60 Ending May 8, 2017 milestone Apr 28, 2017
@chriskacerguis chriskacerguis self-assigned this Apr 28, 2017
@miq-bot
Copy link
Member

miq-bot commented Apr 28, 2017

Checked commit AllenBW@cf500bf with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
0 files checked, 0 offenses detected
Everything looks fine. 🍪

@chriskacerguis chriskacerguis merged commit e147a2f into ManageIQ:master Apr 29, 2017
@AllenBW AllenBW deleted the BZ/#1446405 branch May 1, 2017 13:10
simaishi pushed a commit that referenced this pull request May 1, 2017
BZ#1446405-Update skinning documentation
(cherry picked from commit e147a2f)

https://bugzilla.redhat.com/show_bug.cgi?id=1447051
@simaishi
Copy link
Contributor

simaishi commented May 1, 2017

Fine backport details:

$ git log -1
commit 70b4c1291d3807aa6ff3ac2f69db32b068fd5201
Author: Chris Kacerguis <chriskacerguis@users.noreply.github.com>
Date:   Sat Apr 29 16:07:49 2017 -0500

    Merge pull request #727 from AllenBW/BZ/#1446405
    
    BZ#1446405-Update skinning documentation
    (cherry picked from commit e147a2f5f4f5d7a0483351378834c693071aa7c3)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1447051

@simaishi
Copy link
Contributor

@AllenBW @chriskacerguis this change is requested to be backported to Euwe branch. Would that be a problem?

@AllenBW
Copy link
Member Author

AllenBW commented Jun 13, 2017

@simaishi nope! they picked a good one to backport with one exception, euwe has an additional mention of the image https://github.com/manageiq/manageiq-ui-service/blob/6a3dc554677c1294f4662112aff29e0b20ebe1af/client/app/states/requests/orders/orders.html#L14

SOOOOO to catch all maybe we should do a euwe specific pr? @chriskacerguis thoughts?

@chriskacerguis
Copy link
Contributor

Yeah, I think a euwe specific one makes sense.

AllenBW added a commit to AllenBW/manageiq-ui-service that referenced this pull request Jun 13, 2017
@simaishi
Copy link
Contributor

Backported to Euwe via #817

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