-
Notifications
You must be signed in to change notification settings - Fork 3k
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
feat(ui/data-contract): Data contract UI under Validation Tab #10625
Merged
jayacryl
merged 32 commits into
datahub-project:master
from
amit-apptware:ag--datacontract-tab
Jun 19, 2024
Merged
Changes from 24 commits
Commits
Show all changes
32 commits
Select commit
Hold shift + click to select a range
bdd9667
feat(graphql) data contract resolvers for graphql
jayacryl a240d12
feat(data-contract): Add initial file for data contract on validation…
amit-apptware 2704ca7
feat(data-contract): add datacontract tab file and its graphql files
amit-apptware 6785fc9
feat(data-contract): add modal files to create contract
amit-apptware e89e384
spotless
jayacryl 10c7568
wired contracts to resolver
jayacryl 292a936
added schema file
jayacryl 437b63c
Merge branch 'master' into jp--datacontract-tab
jayacryl 3a6b3ef
feat(ui/data-contract): remove acryl file and use datahub component f…
amit-apptware 23669a0
Merge remote-tracking branch 'upstream/jp--datacontract-tab' into ag-…
amit-apptware 3882ece
feat(ui/data-contract): remove other acryl files and create OSS files
amit-apptware 25536da
feat(ui/data-contract): add condtional rendering the assertion list i…
amit-apptware 0cb37e2
feat(ui/data-contract): removed unwanted code for data contract
amit-apptware 2d872c8
feat(ui/data-contract): add datacontract image for assertion list
amit-apptware 7915c5f
feat(ui/data-contract): add contract icon assigned to assertion
amit-apptware 0555c61
feat(ui/data-contract): change graphql request to update the status a…
amit-apptware 21e148a
feat(ui/data-contract): removed unwanted code
amit-apptware 8c7d796
feat(ui/data-contract): Merge latest code from master
amit-apptware 0bd62b1
feat(ui/data-contract): resolved PR comments
amit-apptware b1278f9
feat(ui/data-contract): resolved PR comments
amit-apptware 364d508
feat(ui/data-contract): resolved PR comments
amit-apptware c8e71e7
feat(ui/data-contract): resolved PR comments
amit-apptware 036cdf3
feat(ui/data-contract): create util function for the volume
amit-apptware abadacb
merge master into feature
amit-apptware f74169a
feat(feature-flags): data contract feature flag
jayacryl f74b7d9
feat(ui/data-contracct): remove yarn lock file changes
amit-apptware b598719
nit fix
jayacryl 26c99d7
disable data contracts by default
jayacryl 8761259
WIP: a240d125adb feat(data-contract): Add initial file for data contr…
amit-apptware 5dd3125
feat(ui/data-contract): fix lint issues
amit-apptware 650ac95
feat(ui/data-contract): fix lint issues
amit-apptware 2b407d2
Merge branch 'master' into ag--datacontract-tab
amit-apptware File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
This file contains 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
31 changes: 31 additions & 0 deletions
31
...ub-web-react/src/app/entity/shared/tabs/Dataset/Validations/FieldAssertionDescription.tsx
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,31 @@ | ||
import React from 'react'; | ||
import { Typography } from 'antd'; | ||
import { FieldAssertionInfo } from '../../../../../../types.generated'; | ||
import { | ||
getFieldDescription, | ||
getFieldOperatorDescription, | ||
getFieldParametersDescription, | ||
getFieldTransformDescription, | ||
} from './fieldDescriptionUtils'; | ||
|
||
type Props = { | ||
assertionInfo: FieldAssertionInfo; | ||
}; | ||
|
||
/** | ||
* A human-readable description of a Field Assertion. | ||
*/ | ||
export const FieldAssertionDescription = ({ assertionInfo }: Props) => { | ||
const field = getFieldDescription(assertionInfo); | ||
const operator = getFieldOperatorDescription(assertionInfo); | ||
const transform = getFieldTransformDescription(assertionInfo); | ||
const parameters = getFieldParametersDescription(assertionInfo); | ||
|
||
return ( | ||
<Typography.Text> | ||
{transform} | ||
{transform ? ' of ' : ''} | ||
<Typography.Text code>{field}</Typography.Text> {operator} {parameters} | ||
</Typography.Text> | ||
); | ||
}; |
16 changes: 16 additions & 0 deletions
16
datahub-web-react/src/app/entity/shared/tabs/Dataset/Validations/SqlAssertionDescription.tsx
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,16 @@ | ||
import React from 'react'; | ||
import { Typography } from 'antd'; | ||
import { AssertionInfo } from '../../../../../../types.generated'; | ||
|
||
type Props = { | ||
assertionInfo: AssertionInfo; | ||
}; | ||
|
||
/** | ||
* A human-readable description of a SQL Assertion. | ||
*/ | ||
export const SqlAssertionDescription = ({ assertionInfo }: Props) => { | ||
const { description } = assertionInfo; | ||
|
||
return <Typography.Text>{description}</Typography.Text>; | ||
}; |
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice, for context john - Amit did a minimal set of changes to make the existing assertion list in OSS selectable (check/uncheck option). This will work for OSS until we get to porting over our acryl assertion list code.