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

Rename: ChargebackRateDetailCurrency -> Currency #19538

Conversation

lpichler
Copy link
Contributor

  • renaming model and its usages (In specs)
  • TODO: rename column:
    ChargebackRateDetail#chargeback_rate_detail_currency_id -> ChargebackRateDetail#currency_id

Links

related (merged) PRs:

@miq-bot assign @kbrock

@miq-bot add_label technical debt

@chessbyte
Copy link
Member

ChargebackRate

Copy link
Member

Choose a reason for hiding this comment

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

This blank line was here intentionally, but I can't remember why I split the list 😆 ...can you leave it, and also keep Currency in the upper section?

@lpichler lpichler force-pushed the rename_ChargebackRateDetailCurrencyToCurrency branch 3 times, most recently from c8b7e00 to 1787fcd Compare November 21, 2019 09:23
@lpichler lpichler changed the title Rename: ChargebackRateDetailCurrency -> Currency [WIP] Rename: ChargebackRateDetailCurrency -> Currency Nov 21, 2019
@miq-bot miq-bot added the wip label Nov 21, 2019
@Fryguy
Copy link
Member

Fryguy commented Nov 21, 2019

#19542 was merged.

@lpichler lpichler force-pushed the rename_ChargebackRateDetailCurrencyToCurrency branch from 1787fcd to 37bae1c Compare November 22, 2019 10:42
@lpichler lpichler force-pushed the rename_ChargebackRateDetailCurrencyToCurrency branch from 37bae1c to 1a89817 Compare November 28, 2019 10:38
@lpichler lpichler changed the title [WIP] Rename: ChargebackRateDetailCurrency -> Currency Rename: ChargebackRateDetailCurrency -> Currency Nov 28, 2019
@miq-bot miq-bot added unmergeable and removed wip labels Dec 2, 2019
@lpichler lpichler force-pushed the rename_ChargebackRateDetailCurrencyToCurrency branch from 1a89817 to 8137c73 Compare December 4, 2019 16:53
@miq-bot
Copy link
Member

miq-bot commented Dec 4, 2019

Checked commit lpichler@8137c73 with ruby 2.5.5, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0
9 files checked, 0 offenses detected
Everything looks fine. 🍪

@lpichler
Copy link
Contributor Author

lpichler commented Dec 9, 2019

this can be merged as ManageIQ/manageiq-cross_repo-tests#19 is not failing already (just one unrelated repo)

@kbrock kbrock merged commit 8d2034d into ManageIQ:master Dec 9, 2019
@kbrock kbrock added this to the Sprint 126 Ending Dec 9, 2019 milestone Dec 9, 2019
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.

5 participants