Skip to content

Consistency checks for blob documents#11392

Open
ester-nl wants to merge 7 commits into
mendix:developmentfrom
ester-nl:appext/1389-blob-consistency-checks
Open

Consistency checks for blob documents#11392
ester-nl wants to merge 7 commits into
mendix:developmentfrom
ester-nl:appext/1389-blob-consistency-checks

Conversation

@ester-nl

Copy link
Copy Markdown
Contributor

No description provided.


This error will also show if one of your checks throw an unexpected exception. In order to know what the exception was, you can check the Mendix logs.

It is important to remember that you are not allowed to call save operations on the model api when the consistency checks are running. An error will throw and you will see the generic error in the Errors Pane again.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

There's a note saying this at the beginning of the page. Is this intentional?


<div style={{ marginTop: 16 }}>
<button onClick={selectConstant}>Select Config Constant (must be String)</button>
</div>a

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

there's a random "a" after the diff block

);

const constant = constants.find(c => c.$ID === data.configConstant!.id) ?? null;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

if (constant?.excluded) {
    result.push({
        errorCode: DOC_EXCLUDED,
        errorDescription: "Referenced document is excluded",
        severity: "error",
        elementText: "Config Constant",
        propertyName: "configConstant"
    });
}

Please add this to give the users a more robust example by default.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants