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

feat(mikro-orm): rename all occurrences of connection #1746

Conversation

derevnjuk
Copy link
Contributor

@derevnjuk derevnjuk commented Jan 30, 2022

Information

Type Breaking change
Feature No

As discussed in #1742 (comment), we should rename all occurrences of connection to satisfy the naming convention in MikroOrm.

Deprecations

  • To register a new request context for the particular MikroOrm instance, use TransactionOptions.contextName since TransactionOptions.connectionName is deprecated.
  • The DbContext::getContext method is deprecated, use DbContext::entries instead.
  • The MikroOrmRegistry::createConnection method is deprecated, instead use MikroOrmRegistry::register to register a particular MikroOrm instance.
  • The MikroOrmRegistry::closeConnections method is deprecated, instead use MikroOrmRegistry::clear to dispose registered MikroOrm instances and clear the registry.

Todos

  • Tests
  • Coverage
  • Example
  • Documentation

closes #1741

@derevnjuk derevnjuk self-assigned this Jan 30, 2022
@derevnjuk derevnjuk force-pushed the feat-#1741/rename-all-occurrences-of-connection branch from 74365ac to 66dfd6e Compare January 30, 2022 11:26
@derevnjuk derevnjuk marked this pull request as ready for review January 30, 2022 11:29
@derevnjuk derevnjuk requested a review from Romakita January 30, 2022 11:29
@derevnjuk derevnjuk force-pushed the feat-#1741/rename-all-occurrences-of-connection branch from 66dfd6e to 0e3f587 Compare January 30, 2022 12:03
@derevnjuk derevnjuk marked this pull request as draft January 30, 2022 12:32
@derevnjuk derevnjuk force-pushed the feat-#1741/rename-all-occurrences-of-connection branch from e95f5c9 to ea4e2c0 Compare January 30, 2022 12:40
@derevnjuk derevnjuk marked this pull request as ready for review January 30, 2022 12:40
@derevnjuk derevnjuk force-pushed the feat-#1741/rename-all-occurrences-of-connection branch from ea4e2c0 to 879bee9 Compare January 30, 2022 13:33
@Romakita Romakita merged commit a7b8645 into tsedio:production Jan 30, 2022
@Romakita
Copy link
Collaborator

🎉 This PR is included in version 6.100.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@derevnjuk derevnjuk deleted the feat-#1741/rename-all-occurrences-of-connection branch January 30, 2022 19:39
@Romakita
Copy link
Collaborator

🎉 This PR is included in version 7.0.0-beta.4 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CHANGE] Create Orm alias decorator for MikroORM
2 participants