FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Added an `Aggregation` section to the `tips` documentation page. by somada141 · Pull Request #168 · graphql-python/graphene-sqlalchemy · GitHub

Added an Aggregation section to the tips documentation page. - #168

Open
somada141 wants to merge 1 commit into
graphql-python:masterfrom
somada141:feature/doc-tips-aggregation
Open

Added an Aggregation section to the tips documentation page.#168
somada141 wants to merge 1 commit into
graphql-python:masterfrom
somada141:feature/doc-tips-aggregation

Conversation

Copy link
Copy Markdown
Contributor

Added an Aggregation section to tips.rst to show how to retrieve the session out of info and perform an aggregation query wrapping the results in a custom type and exposing it to the schema.

Copy link
Copy Markdown

Coverage remained the same at 91.854% when pulling 28ae4ff on somada141:feature/doc-tips-aggregation into 33d5b74 on graphql-python:master.

2 similar comments

Copy link
Copy Markdown

Coverage remained the same at 91.854% when pulling 28ae4ff on somada141:feature/doc-tips-aggregation into 33d5b74 on graphql-python:master.

Copy link
Copy Markdown

Coverage remained the same at 91.854% when pulling 28ae4ff on somada141:feature/doc-tips-aggregation into 33d5b74 on graphql-python:master.

Copy link
Copy Markdown
Collaborator

@somada141 Nice example, but is it really related to graphene ?

The "tips" is "how to retrieve easily the sqlalchemy session from graphene context", but all the "what you can do with the query" is more sqlalchemy stuff than graphene stuff, no ?
So maybe most of the code you wrote should go as example in the appropriated directory instead of in the tips ?

(we can merge that this way and create a fully functional aggragation example later)

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL