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

Add ReadModel migrateAll method #1393

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

gonzalojaubert
Copy link
Contributor

Description

Add ReadModel migrateAll method

Checks

  • Project Builds
  • Project passes tests and checks
  • Updated documentation accordingly

@gonzalojaubert gonzalojaubert marked this pull request as draft April 4, 2023 13:10
@what-the-diff
Copy link

what-the-diff bot commented Apr 4, 2023

PR Summary

  • Added migrateAll method to ReadModelSchemaMigrator
    This update introduces a new method called migrateAll which allows migration of all read models of a specific type in one step.
  • Updated read-model-schema-migrator tests
    New tests have been added to verify the functionality of the migrateAll method.
  • Created migration command for integration testing
    A new command has been created for integration testing purposes, which utilizes the migrateAll method.
  • Created example migration and schema versions for CartReadModel
    An example has been provided to demonstrate the usage of the updated migrateAll method with the CartReadModel.
  • Updated data migrations documentation
    The documentation has been updated to include more information about the new migrateAll method.

@ghost
Copy link

ghost commented Apr 4, 2023

👇 Click on the image for a new way to code review

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map legend

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.

1 participant