Skip to content

Conversation

@jackwaudby
Copy link
Contributor

@jackwaudby jackwaudby commented Nov 11, 2024

In 5.26 SeedProvider was deprecated in favour of DatabaseSeedProvider. In 5 LTS SeedProvider and DatabaseSeedProvider will co-exist. This PR adds DatabaseSeedProvider to the documentation with the relevant deprecated/introduction labels.

@jackwaudby jackwaudby force-pushed the 5-database-seed-provider branch from 89a5f2b to 9b43a10 Compare November 11, 2024 10:51
@jackwaudby jackwaudby changed the title Update implement custom seed providers Introduce DatabaseSeedProvider Nov 11, 2024
@NataliaIvakina NataliaIvakina self-requested a review November 11, 2024 10:59
Copy link
Contributor

@NataliaIvakina NataliaIvakina left a comment

Choose a reason for hiding this comment

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

Just a few editorial suggestions. Otherwise, looks good. Thank you, @jackwaudby!

jackwaudby and others added 7 commits November 13, 2024 10:59
Co-authored-by: NataliaIvakina <82437520+NataliaIvakina@users.noreply.github.com>
Co-authored-by: NataliaIvakina <82437520+NataliaIvakina@users.noreply.github.com>
Co-authored-by: NataliaIvakina <82437520+NataliaIvakina@users.noreply.github.com>
Co-authored-by: NataliaIvakina <82437520+NataliaIvakina@users.noreply.github.com>
Co-authored-by: NataliaIvakina <82437520+NataliaIvakina@users.noreply.github.com>
Co-authored-by: NataliaIvakina <82437520+NataliaIvakina@users.noreply.github.com>
Co-authored-by: NataliaIvakina <82437520+NataliaIvakina@users.noreply.github.com>
@neo-technology-commit-status-publisher
Copy link
Collaborator

neo-technology-commit-status-publisher commented Nov 13, 2024

Thanks for the documentation updates.

The preview documentation has now been torn down - reopening this PR will republish it.

@jackwaudby jackwaudby merged commit f7acfc5 into dev Nov 13, 2024
5 checks passed
@NataliaIvakina NataliaIvakina deleted the 5-database-seed-provider branch November 13, 2024 11:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants