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: Add initial documentation. #263

Merged
merged 4 commits into from
Aug 30, 2023

Conversation

kinyoklion
Copy link
Member

No description provided.

* Report a failed migration operation from `readNew`, `readOld`, `writeNew` or `writeOld`.
*
* ```
* readNew: async () => {
Copy link
Member Author

Choose a reason for hiding this comment

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

This is the same sample, but I think it makes things easier for customers.

@kinyoklion kinyoklion changed the title doc: Add initial documentation. core: Add initial documentation. Aug 30, 2023
* TKTK
* Implementation which provides a write to the "new" source.
*
* Users are required to provide two different write methods -- one to write to the old migration source, and one to
Copy link
Member Author

Choose a reason for hiding this comment

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

There is a bit of duplication here, again because it makes finding the information a bit easier.

@kinyoklion kinyoklion marked this pull request as ready for review August 30, 2023 21:00
@kinyoklion kinyoklion changed the title core: Add initial documentation. chore: Add initial documentation. Aug 30, 2023
@louis-launchdarkly
Copy link
Contributor

Exciting that this is coming together!

@kinyoklion kinyoklion merged commit e12068a into feat/node-migrations Aug 30, 2023
13 of 14 checks passed
@kinyoklion kinyoklion deleted the rlamb/migration-docs-first-pass branch August 30, 2023 22:32
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.

2 participants