graphql / graphql/dataloader

[REQUEST] ESM version of Dataloader

Open
#379 2 comments 5 reactions 0 assignees View on GitHub
enhancement
Dominant language
JavaScript
Stars
13.4k
Forks
518
Avg merge
2d 4h
Merged PRs (30d)
2

Description

## What problem are you trying to solve?
The library is 6 years old and could benefit from using modern ES features, plus ESM is easier to tree-shake. I believe I had some TypeScript issues as well -- dataloader is built using Flow so it requires more maintenance.

## Describe the solution you'd like

Implementation: https://github.com/macalinao/grill/tree/master/packages/dataloader-es

I've also ported over all tests. I also exclusively use `setTimeout` instead of checking for environment-- not sure if this will cause issues.

## Describe alternatives you've considered

N/A

## Additional context

https://github.com/macalinao/grill/tree/master/packages/dataloader-es

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.