Skip to content

Commit 01d33b1

Browse files
--wip-- [skip ci]
1 parent 20ca1d8 commit 01d33b1

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

66 files changed

+585
-299
lines changed

datahub-web-react/src/alchemy-components/components/Icon/Icon.tsx

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,8 @@ export const iconDefaults: IconPropsDefaults = {
1717
tooltipText: '',
1818
};
1919

20+
const ICON_NAMES = getIconNames();
21+
2022
export const Icon = ({
2123
icon,
2224
source = iconDefaults.source,
@@ -29,7 +31,7 @@ export const Icon = ({
2931
tooltipText,
3032
...props
3133
}: IconProps) => {
32-
const { filled, outlined } = getIconNames();
34+
const { filled, outlined } = ICON_NAMES;
3335
const { theme } = useCustomTheme();
3436

3537
// Return early if no icon is provided

datahub-web-react/src/alchemy-components/components/Input/Input.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ import { getInputType } from '@components/components/Input/utils';
1919
export const inputDefaults: InputProps = {
2020
value: '',
2121
setValue: () => {},
22-
label: 'Label',
22+
label: '',
2323
placeholder: 'Placeholder',
2424
error: '',
2525
warning: '',

datahub-web-react/src/alchemy-components/components/TextArea/TextArea.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ import {
1414
import { TextAreaProps } from '@components/components/TextArea/types';
1515

1616
export const textAreaDefaults: TextAreaProps = {
17-
label: 'Label',
17+
label: '',
1818
placeholder: 'Placeholder',
1919
error: '',
2020
warning: '',

datahub-web-react/src/app/ingestV2/source/builder/IngestionSourceBuilderModal.tsx

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,11 +10,12 @@ import { CreateScheduleStep } from '@app/ingestV2/source/builder/CreateScheduleS
1010
import { DefineRecipeStep } from '@app/ingestV2/source/builder/DefineRecipeStep';
1111
import { NameSourceStep } from '@app/ingestV2/source/builder/NameSourceStep';
1212
import { SelectTemplateStep } from '@app/ingestV2/source/builder/SelectTemplateStep';
13-
import sourcesJson from '@app/ingestV2/source/builder/sources.json';
1413
import { SourceBuilderState, StepProps } from '@app/ingestV2/source/builder/types';
1514

1615
import { IngestionSource } from '@types';
1716

17+
import { useIngestionSources } from './useIngestionSources';
18+
1819
const StepsContainer = styled.div`
1920
margin-right: 20px;
2021
margin-left: 20px;
@@ -87,7 +88,7 @@ export const IngestionSourceBuilderModal = ({
8788
},
8889
});
8990

90-
const ingestionSources = JSON.parse(JSON.stringify(sourcesJson)); // TODO: replace with call to server once we have access to dynamic list of sources
91+
const { ingestionSources } = useIngestionSources();
9192

9293
const sendAnalyticsStepViewedEvent = useCallback(
9394
(step: IngestionSourceBuilderStep) => {

datahub-web-react/src/app/ingestV2/source/builder/RecipeForm/azure.ts

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ import { validateURL } from '@app/ingestV2/source/utils';
44
export const AZURE_CLIENT_ID: RecipeField = {
55
name: 'client_id',
66
label: 'Client ID',
7+
helper: 'Application ID from Azure AD',
78
tooltip: 'Application ID. Found in your app registration on Azure AD Portal',
89
type: FieldType.TEXT,
910
fieldPath: 'source.config.client_id',
@@ -15,6 +16,7 @@ export const AZURE_CLIENT_ID: RecipeField = {
1516
export const AZURE_TENANT_ID: RecipeField = {
1617
name: 'tenant_id',
1718
label: 'Tenant ID',
19+
helper: 'Directory ID from Azure AD',
1820
tooltip: 'Directory ID. Found in your app registration on Azure AD Portal',
1921
type: FieldType.TEXT,
2022
fieldPath: 'source.config.tenant_id',
@@ -26,6 +28,7 @@ export const AZURE_TENANT_ID: RecipeField = {
2628
export const AZURE_CLIENT_SECRET: RecipeField = {
2729
name: 'client_secret',
2830
label: 'Client Secret',
31+
helper: 'Azure client secret',
2932
tooltip: 'The Azure client secret.',
3033
type: FieldType.SECRET,
3134
fieldPath: 'source.config.client_secret',
@@ -37,6 +40,7 @@ export const AZURE_CLIENT_SECRET: RecipeField = {
3740
export const AZURE_REDIRECT_URL: RecipeField = {
3841
name: 'redirect',
3942
label: 'Redirect URL',
43+
helper: 'Redirect URL from Azure AD',
4044
tooltip: 'Redirect URL. Found in your app registration on Azure AD Portal.',
4145
type: FieldType.TEXT,
4246
fieldPath: 'source.config.redirect',
@@ -48,6 +52,7 @@ export const AZURE_REDIRECT_URL: RecipeField = {
4852
export const AZURE_AUTHORITY_URL: RecipeField = {
4953
name: 'authority',
5054
label: 'Authority URL',
55+
helper: 'URL for MSAL token requests',
5156
tooltip: 'Is a URL that indicates a directory that MSAL can request tokens from..',
5257
type: FieldType.TEXT,
5358
fieldPath: 'source.config.authority',
@@ -59,6 +64,7 @@ export const AZURE_AUTHORITY_URL: RecipeField = {
5964
export const AZURE_TOKEN_URL: RecipeField = {
6065
name: 'token_url',
6166
label: 'Token URL',
67+
helper: 'URL to acquire Azure tokens',
6268
tooltip:
6369
'The token URL that acquires a token from Azure AD for authorizing requests. This source will only work with v1.0 endpoint.',
6470
type: FieldType.TEXT,
@@ -71,6 +77,7 @@ export const AZURE_TOKEN_URL: RecipeField = {
7177
export const AZURE_GRAPH_URL: RecipeField = {
7278
name: 'graph_url',
7379
label: 'Graph URL',
80+
helper: 'Microsoft Graph API endpoint',
7481
tooltip: 'Microsoft Graph API endpoint',
7582
type: FieldType.TEXT,
7683
fieldPath: 'source.config.graph_url',
@@ -82,6 +89,7 @@ export const AZURE_GRAPH_URL: RecipeField = {
8289
export const AZURE_INGEST_USERS: RecipeField = {
8390
name: 'ingest_users',
8491
label: 'Ingest Users',
92+
helper: 'Ingest users from Azure AD',
8593
tooltip: 'Flag to determine whether to ingest users from Azure AD or not.',
8694
type: FieldType.BOOLEAN,
8795
fieldPath: 'source.config.ingest_users',
@@ -91,6 +99,7 @@ export const AZURE_INGEST_USERS: RecipeField = {
9199
export const AZURE_INGEST_GROUPS: RecipeField = {
92100
name: 'ingest_groups',
93101
label: 'Ingest Groups',
102+
helper: 'Ingest groups from Azure AD',
94103
tooltip: 'Flag to determine whether to ingest groups from Azure AD or not.',
95104
type: FieldType.BOOLEAN,
96105
fieldPath: 'source.config.ingest_groups',
@@ -101,6 +110,7 @@ const schemaAllowFieldPathGroup = 'source.config.groups_pattern.allow';
101110
export const GROUP_ALLOW: RecipeField = {
102111
name: 'groups.allow',
103112
label: 'Allow Patterns',
113+
helper: 'Include specific groups',
104114
tooltip:
105115
'Only include specific schemas by providing the name of a schema, or a regular expression (regex) to include specific schemas. If not provided, all schemas inside allowed databases will be included.',
106116
placeholder: 'group_pattern',
@@ -117,6 +127,7 @@ const schemaDenyFieldPathGroup = 'source.config.groups_pattern.deny';
117127
export const GROUP_DENY: RecipeField = {
118128
name: 'groups.deny',
119129
label: 'Deny Patterns',
130+
helper: 'Exclude specific groups',
120131
tooltip:
121132
'Exclude specific schemas by providing the name of a schema, or a regular expression (regex). If not provided, all schemas inside allowed databases will be included. Deny patterns always take precedence over allow patterns.',
122133
placeholder: 'user_pattern',
@@ -133,6 +144,7 @@ const schemaAllowFieldPathUser = 'source.config.users_pattern.allow';
133144
export const USER_ALLOW: RecipeField = {
134145
name: 'user.allow',
135146
label: 'Allow Patterns',
147+
helper: 'Include specific users',
136148
tooltip:
137149
'Exclude specific schemas by providing the name of a schema, or a regular expression (regex). If not provided, all schemas inside allowed databases will be included. Deny patterns always take precedence over allow patterns.',
138150
placeholder: 'user_pattern',
@@ -149,6 +161,7 @@ const schemaDenyFieldPathUser = 'source.config.users_pattern.deny';
149161
export const USER_DENY: RecipeField = {
150162
name: 'user.deny',
151163
label: 'Deny Patterns',
164+
helper: 'Exclude specific users',
152165
tooltip:
153166
'Exclude specific schemas by providing the name of a schema, or a regular expression (regex). If not provided, all schemas inside allowed databases will be included. Deny patterns always take precedence over allow patterns.',
154167
placeholder: 'user_pattern',
@@ -164,6 +177,7 @@ export const USER_DENY: RecipeField = {
164177
export const SKIP_USERS_WITHOUT_GROUP: RecipeField = {
165178
name: 'skip_users_without_a_group',
166179
label: 'Skip users without group',
180+
helper: 'Skip users without group',
167181
tooltip: 'Whether to skip users without group from Okta.',
168182
type: FieldType.BOOLEAN,
169183
fieldPath: 'source.config.skip_users_without_a_group',

datahub-web-react/src/app/ingestV2/source/builder/RecipeForm/bigquery.ts

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ import { FieldType, RecipeField } from '@app/ingestV2/source/builder/RecipeForm/
33
export const BIGQUERY_PROJECT_ID: RecipeField = {
44
name: 'project_id',
55
label: 'Project ID',
6+
helper: 'Project ID for queries',
67
tooltip: 'Project ID where you have rights to run queries and create tables.',
78
placeholder: 'my-project-123',
89
type: FieldType.TEXT,
@@ -14,6 +15,7 @@ export const BIGQUERY_PROJECT_ID: RecipeField = {
1415
export const BIGQUERY_CREDENTIAL_PROJECT_ID: RecipeField = {
1516
name: 'credential.project_id',
1617
label: 'Credentials Project ID',
18+
helper: 'Project ID from service account',
1719
tooltip: "The Project ID, which can be found in your service account's JSON Key (project_id)",
1820
placeholder: 'my-project-123',
1921
type: FieldType.TEXT,
@@ -24,6 +26,7 @@ export const BIGQUERY_CREDENTIAL_PROJECT_ID: RecipeField = {
2426
export const BIGQUERY_PRIVATE_KEY_ID: RecipeField = {
2527
name: 'credential.private_key_id',
2628
label: 'Private Key Id',
29+
helper: 'Private Key ID from service account',
2730
tooltip: "The Private Key id, which can be found in your service account's JSON Key (private_key_id)",
2831
type: FieldType.SECRET,
2932
fieldPath: 'source.config.credential.private_key_id',
@@ -35,6 +38,7 @@ export const BIGQUERY_PRIVATE_KEY_ID: RecipeField = {
3538
export const BIGQUERY_PRIVATE_KEY: RecipeField = {
3639
name: 'credential.private_key',
3740
label: 'Private Key',
41+
helper: 'Private key from service account',
3842
tooltip: "The Private key, which can be found in your service account's JSON Key (private_key).",
3943
placeholder: '-----BEGIN PRIVATE KEY-----....\n-----END PRIVATE KEY-----',
4044
type: FieldType.SECRET,
@@ -46,6 +50,7 @@ export const BIGQUERY_PRIVATE_KEY: RecipeField = {
4650
export const BIGQUERY_CLIENT_EMAIL: RecipeField = {
4751
name: 'credential.client_email',
4852
label: 'Client Email',
53+
helper: 'Client email from service account',
4954
tooltip: "The Client Email, which can be found in your service account's JSON Key (client_email).",
5055
placeholder: '[email protected]',
5156
type: FieldType.TEXT,
@@ -57,9 +62,10 @@ export const BIGQUERY_CLIENT_EMAIL: RecipeField = {
5762
export const BIGQUERY_CLIENT_ID: RecipeField = {
5863
name: 'credential.client_id',
5964
label: 'Client ID',
65+
helper: 'Client ID from service account',
6066
tooltip: "The Client ID, which can be found in your service account's JSON Key (client_id).",
6167
placeholder: '123456789098765432101',
62-
type: FieldType.TEXTAREA,
68+
type: FieldType.TEXT,
6369
fieldPath: 'source.config.credential.client_id',
6470
rules: null,
6571
required: true,

datahub-web-react/src/app/ingestV2/source/builder/RecipeForm/bigqueryBeta.ts

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ import { FieldType, RecipeField, setListValuesOnRecipe } from '@app/ingestV2/sou
33
export const BIGQUERY_BETA_PROJECT_ID: RecipeField = {
44
name: 'credential.project_id',
55
label: 'Project ID',
6+
helper: 'Project ID from service account',
67
tooltip: "The Project ID, which can be found in your service account's JSON Key (project_id)",
78
placeholder: 'my-project-123',
89
type: FieldType.TEXT,
@@ -15,6 +16,7 @@ const projectIdAllowFieldPath = 'source.config.project_id_pattern.allow';
1516
export const PROJECT_ALLOW: RecipeField = {
1617
name: 'project_id_pattern.allow',
1718
label: 'Allow Patterns',
19+
helper: 'Filter for project IDs',
1820
tooltip: 'Use regex here to filter for project IDs.',
1921
placeholder: '^my_db$',
2022
type: FieldType.LIST,
@@ -30,6 +32,7 @@ const projectIdDenyFieldPath = 'source.config.project_id_pattern.deny';
3032
export const PROJECT_DENY: RecipeField = {
3133
name: 'project_id_pattern.deny',
3234
label: 'Deny Patterns',
35+
helper: 'Filter out project IDs',
3336
tooltip: 'Use regex here to filter for project IDs.',
3437
placeholder: '^my_db$',
3538
type: FieldType.LIST,
@@ -45,6 +48,7 @@ const datasetAllowFieldPath = 'source.config.dataset_pattern.allow';
4548
export const DATASET_ALLOW: RecipeField = {
4649
name: 'dataset_pattern.allow',
4750
label: 'Allow Patterns',
51+
helper: 'Allow specific datasets',
4852
tooltip: 'Use regex here.',
4953
placeholder: '^my_db$',
5054
type: FieldType.LIST,
@@ -60,6 +64,7 @@ const datasetDenyFieldPath = 'source.config.dataset_pattern.deny';
6064
export const DATASET_DENY: RecipeField = {
6165
name: 'dataset_pattern.deny',
6266
label: 'Deny Patterns',
67+
helper: 'Deny specific datasets',
6368
tooltip: 'Use regex here.',
6469
placeholder: '^my_db$',
6570
type: FieldType.LIST,

0 commit comments

Comments
 (0)