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

Naming of middleware #11

Closed
alloy opened this issue May 30, 2018 · 2 comments
Closed

Naming of middleware #11

alloy opened this issue May 30, 2018 · 2 comments

Comments

@alloy
Copy link
Contributor

alloy commented May 30, 2018

Before publishing our timeout middleware to npm, I figured I’d ask if you had a specific naming scheme in mind for middleware? I settled on graphqlTimeoutMiddleware for now: artsy/metaphysics#1068

Thanks for this library btw!

@maticzav
Copy link
Owner

@alloy this is great! I love to hear that you enjoy using the library. There's no official convention, but I settled in Sentry plugin and Forwarding tool with graphql-middleware-nameoflib.

I think we should create a website or at least a collection of the available plugins to make them easier to discover. Or, we could add them to README! What do you think?

@alloy
Copy link
Contributor Author

alloy commented May 30, 2018

I kinda like suggesting the convention in your README through means of adding a list of them all using the same pattern 👍

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

No branches or pull requests

2 participants