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

[Issue-1550] TO golang -- adds orderby= parameter handling to endpoints #1551

Merged
merged 2 commits into from
Nov 16, 2017
Merged

[Issue-1550] TO golang -- adds orderby= parameter handling to endpoints #1551

merged 2 commits into from
Nov 16, 2017

Conversation

dangogh
Copy link
Member

@dangogh dangogh commented Nov 16, 2017

First stab at it -- may need a bit more work.

@asfgit
Copy link
Contributor

asfgit commented Nov 16, 2017

SUCCESS
No test results found.

@asfgit
Copy link
Contributor

asfgit commented Nov 16, 2017

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR-trafficops-test/606/

@asfgit
Copy link
Contributor

asfgit commented Nov 16, 2017

FAILURE
No test results found.
Test FAILed.

@asfgit
Copy link
Contributor

asfgit commented Nov 16, 2017

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/649/
Test FAILed.

@dewrich dewrich merged commit 4d1fa3e into apache:master Nov 16, 2017
@dangogh
Copy link
Member Author

dangogh commented Nov 16, 2017

Fixes #1550

@mitchell852 mitchell852 changed the title [Issue-1550] WIP -- TO golang -- adds orderby= parameter handling to endpoints [Issue-1550] TO golang -- adds orderby= parameter handling to endpoints Nov 27, 2017
@mitchell852 mitchell852 added this to the 2.2.0 milestone Dec 14, 2017
@mitchell852 mitchell852 added this to Completed in TO API Golang Rewrite Dec 20, 2017
@dangogh dangogh deleted the go-orderby branch March 2, 2018 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

4 participants