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

refs #306, apply default cdrs sorting #312

Merged
merged 1 commit into from May 14, 2018

Conversation

Projects
None yet
3 participants
@gigorok
Contributor

gigorok commented May 14, 2018

default_scope is evil. try to prevent yourself from using it

@workgena

This comment has been minimized.

Contributor

workgena commented May 14, 2018

@gigorok, I also changed app/models/cdr_export.rb that time too 😯

@gigorok

This comment has been minimized.

Contributor

gigorok commented May 14, 2018

@workgena, thanks. also should be added sorting explicitly

@workgena

This comment has been minimized.

Contributor

workgena commented May 14, 2018

+ API/Customer/CDR

refs #306, apply default cdrs sorting
default_scope is evil. try to prevent yourself from using it

@gigorok gigorok changed the title from WIP: refs #306, apply default cdrs sorting to refs #306, apply default cdrs sorting May 14, 2018

@dmitry-sinina dmitry-sinina merged commit 7f93e58 into yeti-switch:master May 14, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment