Skip to content
This repository has been archived by the owner on Sep 3, 2021. It is now read-only.

Duplicate query argument fix #332

Merged
merged 42 commits into from
Oct 24, 2019
Merged

Duplicate query argument fix #332

merged 42 commits into from
Oct 24, 2019

Conversation

michaeldgraham
Copy link
Collaborator

This is a followup PR to the recent augmentation rewrite, fixing a recently discovered bug that caused the generation of duplicate query arguments for filter and orderBy.

@codecov-io
Copy link

codecov-io commented Oct 24, 2019

Codecov Report

Merging #332 into master will increase coverage by 0.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #332      +/-   ##
==========================================
+ Coverage   96.26%   96.27%   +0.01%     
==========================================
  Files          23       23              
  Lines        2568     2576       +8     
==========================================
+ Hits         2472     2480       +8     
  Misses         96       96
Impacted Files Coverage Δ
src/augment/input-values.js 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 576a567...241cc62. Read the comment docs.

@johnymontana johnymontana merged commit eec9989 into neo4j-graphql:master Oct 24, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants