Skip to content
This repository was archived by the owner on Nov 10, 2025. It is now read-only.

Import blocks after plugins are loaded.#53

Merged
jsolman merged 6 commits intoplugins-2.10.1from
ImportBlocksAfterPluginsLoad
Mar 14, 2019
Merged

Import blocks after plugins are loaded.#53
jsolman merged 6 commits intoplugins-2.10.1from
ImportBlocksAfterPluginsLoad

Conversation

@jsolman
Copy link
Copy Markdown
Contributor

@jsolman jsolman commented Mar 11, 2019

All persist plugins need to be loaded before blocks start importing.

Depends on neo-project/neo/pull/633

@jsolman jsolman requested review from erikzhang and vncoelho March 11, 2019 23:19
vncoelho
vncoelho previously approved these changes Mar 12, 2019
Copy link
Copy Markdown
Member

@vncoelho vncoelho left a comment

Choose a reason for hiding this comment

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

I agree, the design looks good to me!

But let's wait for @erikzhang to check this new ideas.

shargon
shargon previously approved these changes Mar 13, 2019
@jsolman
Copy link
Copy Markdown
Contributor Author

jsolman commented Mar 13, 2019

This also still references NuGet CI build, thus waiting to merge.

@erikzhang erikzhang dismissed shargon’s stale review March 14, 2019 04:04

CI NuGet reference

@jsolman jsolman changed the base branch from master to plugins-2.10.1 March 14, 2019 18:22
@jsolman jsolman merged commit 01628e0 into plugins-2.10.1 Mar 14, 2019
@jsolman jsolman deleted the ImportBlocksAfterPluginsLoad branch March 16, 2019 02:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants