Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(schema): Use the same options for resolveData hook #2833

Merged
merged 1 commit into from
Oct 26, 2022

Conversation

daffl
Copy link
Member

@daffl daffl commented Oct 25, 2022

This pull request makes sure that the resolveData hook takes the same options as the other resolver hooks and makes sure that it works with custom methods.

Closes #2831

@netlify
Copy link

netlify bot commented Oct 25, 2022

Deploy Preview for feathers-dove ready!

Name Link
🔨 Latest commit 6993d18
🔍 Latest deploy log https://app.netlify.com/sites/feathers-dove/deploys/63582875ccdd5e0008ddaa7d
😎 Deploy Preview https://deploy-preview-2833--feathers-dove.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@daffl daffl merged commit ed3b050 into dove Oct 26, 2022
@daffl daffl deleted the data-resolver-signature branch October 26, 2022 00:54
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

resolveData uses the wrong input parameters
1 participant