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

Add back listing of custom attributes in chargeback #16350

Merged

Conversation

lpichler
Copy link
Contributor

accidently caused by:
9314672#diff-e5855dfa8cc9aeeeb9d13acd1525499eR929

method _custom_details_for(cb_model, {}) is not related to tags as it could looks on first look.

@miq-bot add_label bug, chargeback
@miq-bot assign @gtanzillo

thanks @zeari

@lpichler lpichler closed this Oct 30, 2017
@lpichler lpichler reopened this Oct 30, 2017
@lpichler lpichler closed this Oct 30, 2017
@lpichler lpichler reopened this Oct 30, 2017
@lpichler lpichler closed this Oct 31, 2017
@lpichler lpichler reopened this Oct 31, 2017
@lpichler lpichler closed this Oct 31, 2017
@lpichler lpichler reopened this Oct 31, 2017
@lpichler lpichler changed the title Add back listing of custom attributes in chargeback [WIP] Add back listing of custom attributes in chargeback Nov 2, 2017
@miq-bot miq-bot added the wip label Nov 2, 2017
Copy link
Member

@gtanzillo gtanzillo left a comment

Choose a reason for hiding this comment

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

👍 LGTM. @lpichler can this be unwip'd?

@lpichler lpichler force-pushed the add_custom_attributes_back_to_chargeback branch from 3af16bf to b2a6a30 Compare November 6, 2017 15:08
@lpichler lpichler changed the title [WIP] Add back listing of custom attributes in chargeback Add back listing of custom attributes in chargeback Nov 6, 2017
@lpichler
Copy link
Contributor Author

lpichler commented Nov 6, 2017

@gtanzillo yes, I wanted to add specs 👍

@miq-bot miq-bot removed the wip label Nov 6, 2017
@lpichler
Copy link
Contributor Author

lpichler commented Nov 6, 2017

@miq-bot add_label grapridashvili/yes

@miq-bot
Copy link
Member

miq-bot commented Nov 6, 2017

@lpichler Cannot apply the following label because they are not recognized: grapridashvili/yes

@lpichler lpichler force-pushed the add_custom_attributes_back_to_chargeback branch from b2a6a30 to c6a6d7e Compare November 6, 2017 17:19
@lpichler lpichler force-pushed the add_custom_attributes_back_to_chargeback branch from c6a6d7e to 86f1048 Compare November 7, 2017 13:00
@miq-bot
Copy link
Member

miq-bot commented Nov 7, 2017

Checked commits lpichler/manageiq@a66c25a~...86f1048 with ruby 2.3.3, rubocop 0.47.1, and haml-lint 0.20.0
2 files checked, 0 offenses detected
Everything looks fine. 👍

@lpichler lpichler closed this Nov 7, 2017
@lpichler lpichler reopened this Nov 7, 2017
@gtanzillo gtanzillo added this to the Sprint 73 Ending Nov 13, 2017 milestone Nov 7, 2017
@gtanzillo gtanzillo merged commit 6f1588d into ManageIQ:master Nov 7, 2017
@lpichler lpichler deleted the add_custom_attributes_back_to_chargeback branch November 7, 2017 14:34
simaishi pushed a commit that referenced this pull request Jan 3, 2018
…_chargeback

Add back listing of custom attributes in chargeback
(cherry picked from commit 6f1588d)
@simaishi
Copy link
Contributor

simaishi commented Jan 3, 2018

Gaprindashvili backport details:

$ git log -1
commit 535dc39e522c26333e2dcf8f8e1ffefa6d55a18c
Author: Gregg Tanzillo <[email protected]>
Date:   Tue Nov 7 09:31:10 2017 -0500

    Merge pull request #16350 from lpichler/add_custom_attributes_back_to_chargeback
    
    Add back listing of custom attributes in chargeback
    (cherry picked from commit 6f1588dbae451b98179dce09dd821cce2700d3eb)

@miq-bot
Copy link
Member

miq-bot commented Jan 3, 2018

@lpichler Cannot apply the following label because they are not recognized: grapridashvili/yes

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.

4 participants