Fix minor problems identified in recent pagination work

Bug #1664786 reported by Jim Baker
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
craton
Fix Released
Critical
Ian Cordasco

Bug Description

The pagination code is important, and touching a lot of things. Given tight deadlines, etc, etc, and to ensure it got sequenced in the development, I approved this change. But please fix the minors identified by Thomas Maddox and me:

https://review.openstack.org/#/c/426229/11

Thanks!

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to craton (master)

Fix proposed to branch: master
Review: https://review.openstack.org/434506

Changed in craton:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to craton (master)

Reviewed: https://review.openstack.org/434506
Committed: https://git.openstack.org/cgit/openstack/craton/commit/?id=462c501b13e0510c69a742141f809a21a141685a
Submitter: Jenkins
Branch: master

commit 462c501b13e0510c69a742141f809a21a141685a
Author: Ian Cordasco <email address hidden>
Date: Wed Feb 15 14:40:48 2017 -0600

    Add sort_dir and sort_keys to endpoint schemata

    sort_dir and sort_keys are used in the pagination work. Adding them to
    the schemata allows users to specify them in their queries without
    receiving 400 Bad Request errors.

    Change-Id: If743958afc578be0ffa48c9b0a2670f76855424c
    Partial-Bug: #1664786
    Implements: bp pagination-of-resources

Revision history for this message
Ian Cordasco (icordasc) wrote :

I believe this was fixed. If not we can fix the status.

Changed in craton:
status: In Progress → Fix Committed
Changed in craton:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.