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

fix(tracing): Properly handle tracing fields that resolve an array of lazy values #87

Merged
merged 9 commits into from
Jul 16, 2020

Conversation

rylanc
Copy link
Contributor

@rylanc rylanc commented Jul 10, 2020

This fixes tracing when returning a lazy value from an entity's .resolve_reference method.

@rylanc rylanc changed the title fix(tracing): Properly handle arrays of lazy values fix(tracing): Properly handle tracing fields that resolve arrays of lazy values Jul 10, 2020
@rylanc rylanc changed the title fix(tracing): Properly handle tracing fields that resolve arrays of lazy values fix(tracing): Properly handle tracing fields that resolve an array of lazy values Jul 10, 2020
@rylanc rylanc requested a review from noaelad July 10, 2020 23:17
@rylanc rylanc merged commit a9eed77 into master Jul 16, 2020
@rylanc rylanc deleted the rc-fix-tracing-lazy-arrays branch July 16, 2020 21:31
rylanc pushed a commit that referenced this pull request Jul 16, 2020
## [1.1.3](v1.1.2...v1.1.3) (2020-07-16)

### Bug Fixes

* **tracing:** Properly handle tracing fields that resolve an array of lazy values ([#87](#87)) ([a9eed77](a9eed77))
@rylanc
Copy link
Contributor Author

rylanc commented Jul 16, 2020

🎉 This PR is included in version 1.1.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

mpospelov pushed a commit to toptal/apollo-federation-ruby that referenced this pull request Aug 2, 2021
… lazy values (Gusto#87)

* test: Add failing test

* fix(tracing): Properly handle arrays of lazy values

* refactor: Move EntityTypeResolutionExtension.after_resolve behavior directly into the _entities resolver

* fix: Remove broken require;

* style: Fix linter errors

* test: Add more test cases

* 🎨 Cleanup

* Clarify todo

* style: Fix linter error
mpospelov pushed a commit to toptal/apollo-federation-ruby that referenced this pull request Aug 2, 2021
## [1.1.3](Gusto/apollo-federation-ruby@v1.1.2...v1.1.3) (2020-07-16)

### Bug Fixes

* **tracing:** Properly handle tracing fields that resolve an array of lazy values ([Gusto#87](Gusto#87)) ([a9eed77](Gusto@a9eed77))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants