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

[Breaking][EDT-1607] new property isValueSet on VariableDTO #555

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

evan-mcgeek
Copy link
Contributor

@evan-mcgeek evan-mcgeek commented Dec 27, 2024

This PR adds a new property to the Variable DTO to be a be to check if the value is set on

PR Guidelines

Related tickets

Screenshots

Copy link
Contributor

Coverage Report

Coverage report can be checked at https://stgrafxstudiodevpublic.blob.core.windows.net/sdk/coverage/555/coverage.html

Use PR sdk package

Tarball can be downloaded from https://stgrafxstudiodevpublic.blob.core.windows.net/sdk/dev-packages/555/studio-sdk.tgz

To use in local project, change package.json to use local tarball

"dependencies": {
    "@chili-publish/studio-sdk": "https://stgrafxstudiodevpublic.blob.core.windows.net/sdk/dev-packages/555/studio-sdk.tgz"
}

Copy link
Contributor

Unit Test Results

    1 files    38 suites   42s ⏱️
408 tests 408 ✔️ 0 💤 0
419 runs  419 ✔️ 0 💤 0

Results for commit a17b8cd.

@alexoparii alexoparii added Ready for QA run automated tests and removed Ready for QA run automated tests labels Jan 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants