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.

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

CI Issue fixed and images optimised #413 resolved to #412 #415

Closed
wants to merge 3 commits into from

Conversation

mukeshblackhat
Copy link

@mukeshblackhat mukeshblackhat commented Aug 4, 2024

Created a PR WRT to #412

Proof read identify gaps
Make the build pass
Optimize uploaded images
#413 resolved
/claim #413

@algora-pbc algora-pbc bot mentioned this pull request Aug 4, 2024
@tusharmath
Copy link
Contributor

Can you fix the conflicts?

@mukeshblackhat
Copy link
Author

Can you fix the conflicts?

merge conflicts resolved

2. Start at the root query and begin traversing the graph data structure.
3. For each field in the current node, check if it has a resolver and is not batched. We know if the node contains a resolver if that node has a [`@http`](../docs/directives.md#http-directive) or a [`@grpc`](../docs/directives.md#grpc-directive).
3. For each field in the current node, check if it has a resolver and is not batched. We know if the node contains a resolver if that node has a [`@http`](https://tailcall.run/docs/directives.md#http-directive) or a [`@grpc`](https://tailcall.run/docs/directives.md#grpc-directive).
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
3. For each field in the current node, check if it has a resolver and is not batched. We know if the node contains a resolver if that node has a [`@http`](https://tailcall.run/docs/directives.md#http-directive) or a [`@grpc`](https://tailcall.run/docs/directives.md#grpc-directive).
3. For each field in the current node, check if it has a resolver and is not batched. We know if the node contains a resolver if that node has a [`@http`](/docs/directives.md#http-directive) or a [`@grpc`](/docs/directives.md#grpc-directive).

Dont' use absolute URLs.

@tusharmath tusharmath marked this pull request as draft August 4, 2024 13:35
@tusharmath
Copy link
Contributor

Moving to draft to reduce noise and improve CI efficiency. Once you are ready just mark it as "ready to review". Feel free to give a shoutout on the #contributors channel on Discord if you want immediate attention.

@tusharmath tusharmath closed this Aug 4, 2024
@tusharmath
Copy link
Contributor

Sorry @mukeshblackhat had to close this PR. We are in a bit of a hurry to deploy this blog. Thanks once again for your contributions!

@mukeshblackhat
Copy link
Author

@tusharmath why redirection is not working properly in this file ??

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.

2 participants