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

IHP GraphQL Standalone #1392

Open
wants to merge 60 commits into
base: master
Choose a base branch
from
Open

IHP GraphQL Standalone #1392

wants to merge 60 commits into from

Conversation

mpscholten
Copy link
Member

@mpscholten mpscholten commented Mar 8, 2022 •
edited

@mpscholten
Merge branch 'master' into ihp-graphql
d50f175
@mpscholten
Disabled not useful parts of the navigation
701bad8
@mpscholten
Added input types to graphql schema
e0e322c
@mpscholten
Merge branch 'ihp-graphql' into ihp-graphql-standalone
bc411ef
@mpscholten
Added missing modules to cabal file
466ebb7
@mpscholten
Merge branch 'ihp-graphql' into ihp-graphql-standalone
3e765c6
@mpscholten
Merge branch 'graphql' into ihp-graphql-standalone
276349b
@mpscholten
Initial prototype of the standalone GraphQL edition of IHP
f5045c1
@mpscholten
Added GraphQL Standalone IHP App
52842de
@mpscholten
Moved GraphQL standalone app to it's own repo
0ee64df
@mpscholten
Bundle Graph Explorer JS Assets during nix build
b1a01dc
@mpscholten
Merge remote-tracking branch 'origin/master' into ihp-graphql-standalone
772b333
@mpscholten
Disable nix sandbox for build as we need npm to work
7005c28
@mpscholten
Merge remote-tracking branch 'origin/master' into ihp-graphql-standalone
9aeb054
@mpscholten
Fixed GraphQL queries returning null on empty result set
03ef59f
@mpscholten
Added basic support for live queries
4909b54
@mpscholten
added new module to cabal file
a98adc9
@mpscholten
Merge remote-tracking branch 'origin/master' into ihp-graphql-standalone
525140f
@mpscholten
Enable autoPolicy for user_id column button
1f919ab
@mpscholten
Implemented graphql-ws protocol
29959b7
@mpscholten
Added new modules to cabal file
87d0bbb
@mpscholten
fixed missing jwt dep
efb192e
@mpscholten
Also make jwt available for tests
499a201
@mpscholten
Allow customizing the DATABASE_URL for the migration generator
1f6ff94
@mpscholten
Fixed migration erorr handling not working as expected when the migra…
6422095 
…tion is created and run directly
@mpscholten
Improved dark mode of GraphiQL
31d6ea2
@mpscholten
Fixed Graph Explorer disappear when doubleclicking the GRAPH button
a3a596c
@mpscholten
Fixed GraphiQL Query Variables & Headers tab not always in view
cd8c3fb
@mpscholten
Added user selector to GraphQL explorer
b5f40b8
@mpscholten
Improved placement of copyright
d777d49
@mpscholten
Added favicon and changed titte
25893cf
@mpscholten
Merge remote-tracking branch 'origin/master' into ihp-graphql-standalone
93614d3
@mpscholten
Merge branch 'master' into ihp-graphql-standalone
41793f0
@mpscholten
Removed compiled JS files
ede753b
@mpscholten
GraphQL errors are returned as json
fccb3ba
@mpscholten
Updated graph explorer
30a01be
@mpscholten
Improved error handling for GraphQL queries over WS
a285442
@mpscholten
Fixed error handling
2a88ce7
@mpscholten
Fixed GraphQL query sometimes returning multiple rows
d0f681e
@mpscholten
Fixed "Deselecting last field for query doesn't remove query from exp…
b7fbc80 
…lorer code"

Fixes digitallyinduced/thin-backend#11
Fixes digitallyinduced/thin-backend#16
@mpscholten
Persist GraphiQL query
65cb66b 
Fixes digitallyinduced/thin-backend#2
@mpscholten
Added /api prefix to grapql-ws endpoint
9634c73
@mpscholten
Improved colors of docs in Grapql explorer
294ca42
@mpscholten
Added support for the GraphQL introspection query
e0c4828
@mpscholten
Added new modules to cabal file
2744751
@mpscholten
Removed introspection test
6dc117c
@mpscholten
Fixed generated sql statements causing a cross join which sometimes n…
a67c2f9 
…ot returns the expected results
@mpscholten
Fixed GraphQLWS responses not wrapped in "data" attribute and ids wrong
9494af6
@mpscholten
Added support for __typename selections
685567e
@mpscholten
Fixed row level security accidentally checked for table "createUser" …
b24c439 
…instead of "users" if a mutation is used
@mpscholten
Support __typename also in mutation results
b2284eb
s0kil
s0kil reviewed Mar 27, 2022
View reviewed changes
ihp.nix
];
license = lib.licenses.mit;
preBuild = ''
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This stage currently fails when I check out these changes locally and perform a build:
I have worked around that by escaping nix sandboxing: nix-shell --option build-use-sandbox false since otherwise npm will fail due to no internet access. Is this the best option available for nix+npm, escaping the sandbox?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes known issue. The thin backend binaries are also built without sandbox. We can fix this by using something like https://github.com/svanderburg/node2nix

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@s0kil s0kil s0kil left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@mpscholten @s0kil