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

Pass variables to result function #91

Merged
merged 4 commits into from
Jun 18, 2021

Conversation

ajdean
Copy link
Contributor

@ajdean ajdean commented Jun 18, 2021

Thought it would be handy when using a result function to be able to pass the query variables into the function, so they could be used to help generate the mock data.

Had a go at updating the code to make it so and it seems to work OK. Have also added a test around the functionality.

@insidewhy
Copy link
Owner

This is cool.

@insidewhy
Copy link
Owner

BTW your commit messages are in a mixture of imperative mood and declarative mood :D Most organisations seem to be mandating imperative right now, using that for this repo.

@insidewhy insidewhy merged commit bdc5c99 into insidewhy:main Jun 18, 2021
@ajdean ajdean deleted the feature/vars-to-result-function branch June 18, 2021 03:59
@eli0shin
Copy link

eli0shin commented Jul 2, 2021

@insidewhy Thanks for this package, it makes working with apollo mocks much easier. Any idea when we might see a release or beta with this PR included?

@insidewhy
Copy link
Owner

@ajdean @eli0shin sorry I intended to release 2.0.1 with this two weeks ago, I'll do it today.

@insidewhy
Copy link
Owner

@ajdean @eli0shin Released 2.0.1. I think the built-in mock link should be replaced with wildcard-mock-link so I opened this feature request apollo: apollographql/apollo-feature-requests#298

Would appreciate your reactions or comments on that issue. Thanks again for your contributions!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants