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 missing route for Object Store Objects #3037

Conversation

hstastna
Copy link
Contributor

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

Add missing route for Object Store Objects to enable save_default_search method, fixing an error
occurred when saving a new filter in Adv Search in Storage -> Object Storage -> Object Store Objects
page.
obj_store

Before:
obj_store2

After:
obj_store1

@hstastna
Copy link
Contributor Author

@miq-bot add_label bug, gaprindashvili/yes

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

Add missing route for Object Store Objects to enable save_default_search
method, fixing the error when saving a new filter in Advanced Search in
Storage -> Object Storage -> Object Store Objects page.
@miq-bot
Copy link
Member

miq-bot commented Dec 12, 2017

Checked commit hstastna@d509e90 with ruby 2.3.3, rubocop 0.47.1, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. 👍

Copy link
Member

@romanblanco romanblanco left a comment

Choose a reason for hiding this comment

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

Looks good, fixes the issue 👍

@romanblanco
Copy link
Member

ping @martinpovolny

@martinpovolny martinpovolny merged commit d620e33 into ManageIQ:master Dec 14, 2017
@martinpovolny martinpovolny self-assigned this Dec 14, 2017
@martinpovolny martinpovolny added this to the Sprint 76 Ending Jan 1, 2018 milestone Dec 14, 2017
simaishi pushed a commit that referenced this pull request Dec 14, 2017
…objects_route

Add missing route for Object Store Objects
(cherry picked from commit d620e33)

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

Gaprindashvili backport details:

$ git log -1
commit 12333ca5fffa5455f4e43f7d9eace3db72270e7c
Author: Martin Povolny <mpovolny@redhat.com>
Date:   Thu Dec 14 14:19:37 2017 +0100

    Merge pull request #3037 from hstastna/Adv_search_Cloud_Object_store_objects_route
    
    Add missing route for Object Store Objects
    (cherry picked from commit d620e331dad48b15b71bbce27b56c0024f23d963)
    
    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1526097

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