Repository navigation
MILAB-7194: warn about blocks missing data of this block - #55
Open
AStaroverov wants to merge 3 commits into
Open
AStaroverov wants to merge 3 commits into
AStaroverov wants to merge 3 commits into
Conversation
Contributor
Author
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The block warns when blocks below it use its data and that data is gone or not produced yet.
ui/src/pages/MainPage.vue):<PlBlockDependencies />. It reads the column links the app computes and shows nothing while every link is fine. When another block uses an output this block does not export, values this block no longer has, or data of this block before it has been run, it shows a warning naming those blocks, with advice to add the missing data or run the block; block names open the block.Depends on milaboratory/platforma#1911:
PlBlockDependenciesand column links exist only in that SDK, so CI passes after its release and an SDK bump here.

Confidence Score: 5/5
No new blocking issue was found; merging still depends on the SDK release already called out by AStaroverov.Summary
Adds
<PlBlockDependencies />to warn about missing data used by downstream blocks.Important touched term:
PlBlockDependencies: the SDK component that shows dependency warnings. The page now renders it without ahandleprop and relies on links computed by the desktop app. No code types are added or changed.Reviews (2) · Last reviewed commit: "MILAB-7194: drop the model package from ..." · Reviewed by Greptile