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

chore: show preview features in generator block as auto-completion su… #351

Merged
merged 14 commits into from Jul 27, 2020

Conversation

carmenberndt
Copy link
Contributor

closes #346

@carmenberndt carmenberndt added this to the 2.4.0 milestone Jul 23, 2020
@carmenberndt carmenberndt requested a review from janpio July 23, 2020 14:33
Comment on lines 220 to 241
"generatorPreviewFeatures": [
{
"label": "distinct",
"documentation": "Enables distinct querying capabilities to Prisma Client. It allows you to query for distinct (unique) rows of a model. In other words: The fields you provide in the `distinct` argument will be duplicate free.\nDistinct is only a filter and doesn't change the return type of the `findMany` query.\n\nPlease [share your feedback](https://github.com/prisma/prisma/issues/2835) on how this feature works for you. We are interested in both positive and negative feedback, so we know if this feature is already ready for production! (If encounter any problems, please open a new issue [here](https://github.com/prisma/prisma/issues/new/choose))."
},
{
"label": "middlewares",
"documentation": "Enables the Middlewares API allowing you to hook into the control flow of Prisma Client. Middlewares allows you to manipulate the parameters and interrogate the result of each query.\n\nPlease [share your feedback](https://github.com/prisma/prisma-client-js/issues/770) on how this feature works for you. We are interested in both positive and negative feedback, so we know if this feature is already ready for production! (If encounter any problems, please open a new issue [here](https://github.com/prisma/prisma/issues/new/choose))."
},
{
"label": "connectOrCreate",
"documentation": "While Prisma Client already provides an `upsert` query, which is available on the top-level of the Prisma Client API, we now also support `upsert`s in a *nested* manner. When writing data with Prisma Client, both in `update` and `create`, you can now provide an `upsert` operation on a related row by using this feature flag.\n\nPlease [share](https://github.com/prisma/prisma-client-js/issues/336) your feedback on how this feature works for you. We are interested in both positive and negative feedback, so we know if this feature is already ready for production! (If encounter any problems, please open a new issue here)."
},
{
"label": "transactionApi",
"documentation": "While Prisma already ships transactions within [nested writes](https://www.prisma.io/docs/reference/tools-and-interfaces/prisma-client/relation-queries#nested-writes), there are many use cases, where you might want to perform multiple *unrelated* write operations in a transaction and rollback, if one of them fails.\nBy wrapping your write operations in the new `transaction()` function, you can achieve exactly this!\n\n(Note, that these transactions are not long-running and are executed directly after each other. This is an explicit design decision of Prisma. In case this does not cover your use case yet, please chime in on [GitHub](https://github.com/prisma/prisma-client-js/issues/742)).\nAlternatively, you can store the unresolved promises in variables and pass these to the new transaction function.\n\n\n\nPlease [share your feedback](https://github.com/prisma/prisma-client-js/issues/667) on how this feature works for you. We are interested in both positive and negative feedback, so we know if this feature is already ready for production! (If encounter any problems, please open a new issue [here](https://github.com/prisma/prisma/issues/new/choose))."
},
{
"label": "aggregateApi",
"documentation": "While Prisma Client already provides a `count` query for each model, we now introduce further aggregation functionality for number fields.\nIf your model has a field of type `Int` or `Float`, the following aggregations will be available:\n* sum: Sum up all values of a field for the selected rows.\n* min: Get the minimum value for a field for the selected rows.\n* max: Get the maximum value for a field for the selected rows.\n* avg: Get the average value for a field for the selected rows.\n\nPlease [share](https://github.com/prisma/prisma/issues/2838) your feedback on how this feature works for you. We are interested in both positive and negative feedback, so we know if this feature is already ready for production! (If encounter any problems, please open a new issue [here](https://github.com/prisma/prisma/issues/new/choose))."
}
],
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ohhhhhh... that is interesting. I did not expect that we also suggest the values!

Hmm, quite a bit of maintenance at the same time. But probably worth it? @timsuchanek?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should each of them maybe link to the respective feedback issue that are linked here: prisma/prisma#3108?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nikolasburk Do you mean in addition to the link of the respective issue (e.g. prisma/prisma-client-js#667) or instead of it?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We have new, individual feedback issues listed on prisma/prisma#3108 that people should use to leave feedback.

If those links are indeed clickable, that would be nice.

(@nikolasburk We should also go back to the release notes and update these.)

carmenberndt and others added 5 commits July 27, 2020 11:09
Co-authored-by: Martina Welander <mhwelander@users.noreply.github.com>
Co-authored-by: Martina Welander <mhwelander@users.noreply.github.com>
Co-authored-by: Martina Welander <mhwelander@users.noreply.github.com>
Co-authored-by: Martina Welander <mhwelander@users.noreply.github.com>
Co-authored-by: Martina Welander <mhwelander@users.noreply.github.com>
@carmenberndt carmenberndt merged commit 99b8dec into master Jul 27, 2020
@carmenberndt carmenberndt deleted the autocompletion_show_previewFeature_field branch July 27, 2020 11:51
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

Successfully merging this pull request may close these issues.

Show previewFeatures in generator block as auto-completion suggestion
4 participants