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

Suggestions: basic/advanced/experimental patterns #25

Open
SachaG opened this issue Nov 3, 2019 · 5 comments
Open

Suggestions: basic/advanced/experimental patterns #25

SachaG opened this issue Nov 3, 2019 · 5 comments

Comments

@SachaG
Copy link

SachaG commented Nov 3, 2019

To make it easier to reach consensus on specific patterns, I think it could be useful to tag them according to their "status". What I have in mind would be:

  • Basic: patterns or best practices that nearly everybody agrees on and that should probably be implemented by most GraphQL projects.
  • Advanced: patterns or practices that people agree on, but might not be needed by every project. For example, something like cursor-based pagination.
  • Experimental: patterns that some people find useful, but haven't reached a consensus yet; or maybe go against the GraphQL spec somehow.

Having this "experimental" designation would make it easier to include new patterns even if people don't unanimously approve of them. They could always "graduate" to basic or advanced later on once they become more widespread.

Thoughts?

@RIP21
Copy link
Member

RIP21 commented Nov 3, 2019

Totally agree, that would be nice. Because implementing all the amazing tricks described there is not needed for sure, but having a distinctions like that for newcomers would be amazing.
There was also a plan to build some sort of picker mode. Where teams can pick the rules they are using, and then generate their own "cheatsheet".

PS. Thanks for state of JS ;)

@SachaG
Copy link
Author

SachaG commented Nov 3, 2019

Picker mode would be cool! You could then generate stats to see which patterns are used the most. I expect it would require a lot more back-end complexity to track that data though…

And thanks, stay tuned for the 2019 edition soon (which I should be working on right now instead of doing this actually…)

@nodkz
Copy link
Member

nodkz commented Nov 13, 2019

I agree with basic and advanced but don't feel that experimental name is good enough.

For now, we have the following set of rules. And which tags do you want to set for every rule? Maybe you will found new tags ;)

@nodkz
Copy link
Member

nodkz commented Nov 13, 2019

Anyway tags addition is a great idea 👍

In the future, we can add more tags with different meanings and allow users to filter rules by them. Eg. mutation, query, filtering, auth, args, convention etc.

But first, need to choose good values for pattern tagging.

@SachaG
Copy link
Author

SachaG commented Nov 14, 2019

I agree with basic and advanced but don't feel that experimental name is good enough.

What's the word that other specs use when something is under review or not yet officially accepted? Under consideration? Under review?

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

3 participants