Thank you @arv for the work on #3477. It pointed me in the direction of where to look code wise. I wanted to expand the work that was done to resolve the issue I ran into where defining a destSchema
for a table that isn't referenced in schema.tables
causes an error.
I added test cases for two potential ways to have a missing table. If anyone can think of other scenarios I can gladly write more tests.
My only question about adding this, while I think it improves the DX and will help people to ensure their schema is valid, is there any concern about runtime cost and having createSchema
doing this additional work?
I was trying to use bright to highlight the gleam language. Everything was in place in tm-grammars
to do that, bright just needed to be able to access the newer languages. Updating to the latest release of lighter gives us access to the newer version of tm-grammars
.
I'm not super familiar with lighter, but it also has a pinned exact version of tm-grammars
, currently set to "1.16.2"
. It may be a good idea to relax that requirement so that new languages that get added can be accessed without having to publish newer versions.
Fixes: #203
Preface: This is my first exposure to working with these Node objects on the server side so I'm not entirely sure if what I've done is correct.
I changed the way the incremental responses are sent back to the client. In the issue I noted
for await (const chunk of httpGraphQLResponse.body.asyncIterator) {
body.push(chunk);
}
this await
is stopping the server from incrementally delivering the responses and instead they're all delivered at once.
These changes allow the responses to be streamed back instead of all at once.
The logs in the client now show
what is data? {book: {…}} false 12:22:18
what is data? {book: {…}} false 12:22:19
Where the second book: {}
has the deferred data and is delivered 1 second after the first in which the 1 second is hard coded in my project using this package.
https://github.com/apollographql/apollo-client-nextjs/pull/309/files#r1628079676
As noted in the other PR, GFM actually supports double spaces as line breaks (https://github.github.com/gfm/#hard-line-breaks), who knew!
This uses the other option of \
to indicate a line break so editors don't automatically clean it up and cause issues in the future
Hi, I noticed that the links to useSuspenseQuery
and useFragment
weren't linking to the proper place. It looks like those hooks got moved out of experimental and are stable now. I updated those links. Looks like my editor also cleared out some trailing whitespace, hopefully that is okay.
Resolves #1552
Please see our docs on breaking changes to help!
Type: Breaking change
label)If Yes
, what's the impact:
Please add the corresponding label for change this PR introduces:
Type: Bug
Type: Feature
Type: Documentation
Type: Maintenance
Ran rails app:update
seemed to resolve the CORS issues I was running into
Allow any host so we can run this app on different domains