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

Call miqCheckForChanges in breadcrumbs #6003

Merged
merged 1 commit into from Aug 15, 2019

Conversation

rvsia
Copy link
Contributor

@rvsia rvsia commented Aug 13, 2019

https://bugzilla.redhat.com/show_bug.cgi?id=1725927

Description

  • adds miqCheckForChanges in breadcrumbs

Steps to test:

  1. Go to any form (not DDF)
  2. Change something
  3. Click on breadcrumbs

Before

image

After

image

@miq-bot add_label bug, breadcrumbs, changelog/yes, ivanchuk/no

@miq-bot add_reviewer @Hyperkid123

@rvsia rvsia changed the title Call miqCheckForChanges in breadcrumbs [WIP] Call miqCheckForChanges in breadcrumbs Aug 14, 2019
@miq-bot miq-bot added the wip label Aug 14, 2019
@rvsia rvsia changed the title [WIP] Call miqCheckForChanges in breadcrumbs Call miqCheckForChanges in breadcrumbs Aug 14, 2019
@miq-bot
Copy link
Member

miq-bot commented Aug 14, 2019

Checked commit rvsia@2ece43d with ruby 2.4.6, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0
0 files checked, 0 offenses detected
Everything looks fine. 🏆

@miq-bot miq-bot removed the wip label Aug 14, 2019
@h-kataria h-kataria self-assigned this Aug 15, 2019
@h-kataria h-kataria added this to the Sprint 118 Ending Aug 19, 2019 milestone Aug 15, 2019
@h-kataria h-kataria merged commit 1b4f058 into ManageIQ:master Aug 15, 2019
@simaishi
Copy link
Contributor

@rvsia @h-kataria the BZ is targeted for 5.11.0 but this PR is ivanchuk/no. Is a separate PR needed for ivanchuk branch?

@rvsia
Copy link
Contributor Author

rvsia commented Aug 17, 2019

@simaishi My bad. I didnt notice it. You can backport this.

simaishi pushed a commit that referenced this pull request Aug 19, 2019
@simaishi
Copy link
Contributor

Ivanchuk backport details:

$ git log -1
commit 1441cff22b08b075ade9b706fbe23732dd14d686
Author: Harpreet Kataria <hkataria@redhat.com>
Date:   Thu Aug 15 10:13:34 2019 -0400

    Merge pull request #6003 from rvsia/check-changes-breadcrumbs
    
    Call miqCheckForChanges in breadcrumbs
    
    (cherry picked from commit 1b4f0588d572286b61449459bcef9a9958d47b18)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1725927

@rvsia rvsia deleted the check-changes-breadcrumbs branch September 17, 2019 12:01
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

5 participants