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.

Sign up for GitHub

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

Jump to bottom

Refactor old processOptions/where to use GQL JSON #6072

Merged
merged 1 commit into from
Nov 12, 2015
Merged

Refactor old processOptions/where to use GQL JSON #6072

merged 1 commit into from
Nov 12, 2015

Conversation

ErisDS
Copy link
Member

@ErisDS ErisDS commented Nov 12, 2015

refs #5943

  • no longer assume the options in processOptions are set
  • set where to a new GQL JSON-like statement object
  • rather than setting options, add statements which can be understood by knexify
  • pass the statements through knexify to build the query

@ErisDS
Refactor old processOptions/where to use GQL JSON
4dac01c 
refs TryGhost#5943

- no longer assume the options in processOptions are set
- set where to a new GQL JSON-like statement object
- rather than setting options, add statements which can be understood by knexify
- pass the statements through knexify to build the query
sebgie added a commit that referenced this pull request Nov 12, 2015
@sebgie
Merge pull request #6072 from ErisDS/use-gql
b64a0cc 
Refactor old processOptions/where to use GQL JSON
@sebgie sebgie merged commit b64a0cc into TryGhost:master Nov 12, 2015
@ErisDS ErisDS deleted the use-gql branch November 12, 2015 18:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ErisDS @sebgie