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

Used correct date/time comparison operator for OOTB Policy Events report #12961

Merged
merged 1 commit into from
Dec 2, 2016

Conversation

gtanzillo
Copy link
Member

Updated deprecated date/time comparison operator ">=" to use the correct operator "after"

That operator was removed for date/time fields in #10295

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

@gtanzillo gtanzillo changed the title Used correct date/time cop Used correct date/time comparison operator for OOTB Policy Events report Dec 2, 2016
@chessbyte chessbyte merged commit 3794af1 into ManageIQ:master Dec 2, 2016
@chessbyte chessbyte self-assigned this Dec 2, 2016
@chessbyte chessbyte added this to the Sprint 50 Ending Dec 5, 2016 milestone Dec 2, 2016
chessbyte added a commit that referenced this pull request Dec 2, 2016
Used correct date/time comparison operator for OOTB Policy Events report
(cherry picked from commit 3794af1)

https://bugzilla.redhat.com/show_bug.cgi?id=1400330
@chessbyte
Copy link
Member

Euwe Backport details:

$ git log -1
commit e14f883f5f4d21bfc45354e3afe81bd313d7b66e
Author: Oleg Barenboim <[email protected]>
Date:   Fri Dec 2 12:09:54 2016 -0500

    Merge pull request #12961 from gtanzillo/bz-1400330-7-days-ago
    
    Used correct date/time comparison operator for OOTB Policy Events report
    (cherry picked from commit 3794af123f30ffc5c75207ced0de3123451869ce)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1400330

@chessbyte
Copy link
Member

@simaishi Please create Euwe BZ from https://bugzilla.redhat.com/show_bug.cgi?id=1400330

@simaishi
Copy link
Contributor

simaishi commented Dec 5, 2016

@gtanzillo gtanzillo added the bug label Dec 6, 2016
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.

3 participants