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

docs: Firing custom GraphQL queries using Apollo client #79

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

Pathan-Amaankhan
Copy link
Member

What

  • This PR will add doc for: how to fire custom GraphQL queries using Apollo client.

Related Issue(s):

Testing Instructions

  • Open docs/mutating-data.md and verify that the content is complete and accurate.

Screenshots

SCR-20250219-rcue

Additional Information

  • Currently there is no way to fire query using @snapwp/query package. We will have to expose a query function in @snapwp/query package for the same.

Checklist

  • I have read the Contribution Guidelines.
  • My code is tested to the best of my abilities.
  • My code passes all lints (ESLint, tsc, prettier etc.).
  • My code has detailed inline documentation.
  • I have added unit tests to verify the code works as intended.
  • I have updated the project documentation as needed.
  • I have added a changeset for this PR using npm run changeset.

Copy link

changeset-bot bot commented Feb 19, 2025

⚠️ No Changeset found

Latest commit: 2f7566a

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@Pathan-Amaankhan Pathan-Amaankhan self-assigned this Feb 19, 2025
@Pathan-Amaankhan Pathan-Amaankhan marked this pull request as ready for review February 19, 2025 14:15
@justlevine justlevine requested a review from SH4LIN February 19, 2025 14:55
@justlevine
Copy link
Collaborator

This is perfect - thanks @Pathan-Amaankhan .

I'm not merging this, since it essentially says "if you want to query graphql data, use whatever you want and dont interface with our app".

We'll backfill this as we make progress on improving the DX here for the next sprint.

@justlevine justlevine added status: hold ⏸️ Needs other items to be resolved before work can continue and removed status: hold ⚠️ labels Feb 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: hold ⏸️ Needs other items to be resolved before work can continue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants