Skip to content

Commit ecc2c59

Browse files
committed
[transform] Review feedback: Fix texts.
1 parent ba80b40 commit ecc2c59

File tree

9 files changed

+20
-20
lines changed

9 files changed

+20
-20
lines changed

x-pack/legacy/plugins/transform/public/app/common/navigation.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ import { CLIENT_BASE_PATH, SECTION_SLUG } from '../constants';
1212

1313
/**
1414
* Gets a url for navigating to Discover page.
15-
* @param indexPatternId Index pattern id.
15+
* @param indexPatternId Index pattern ID.
1616
* @param baseUrl Base url.
1717
*/
1818
export function getDiscoverUrl(indexPatternId: string, baseUrl: string): string {

x-pack/legacy/plugins/transform/public/app/common/transform.test.ts

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -7,22 +7,22 @@
77
import { isTransformIdValid } from './transform';
88

99
describe('Transform: isTransformIdValid()', () => {
10-
test('returns true for job id: "good_job-name"', () => {
10+
test('returns true for job ID: "good_job-name"', () => {
1111
expect(isTransformIdValid('good_job-name')).toBe(true);
1212
});
13-
test('returns false for job id: "_bad_job-name"', () => {
13+
test('returns false for job ID: "_bad_job-name"', () => {
1414
expect(isTransformIdValid('_bad_job-name')).toBe(false);
1515
});
16-
test('returns false for job id: "bad_job-name_"', () => {
16+
test('returns false for job ID: "bad_job-name_"', () => {
1717
expect(isTransformIdValid('bad_job-name_')).toBe(false);
1818
});
19-
test('returns false for job id: "-bad_job-name"', () => {
19+
test('returns false for job ID: "-bad_job-name"', () => {
2020
expect(isTransformIdValid('-bad_job-name')).toBe(false);
2121
});
22-
test('returns false for job id: "bad_job-name-"', () => {
22+
test('returns false for job ID: "bad_job-name-"', () => {
2323
expect(isTransformIdValid('bad_job-name-')).toBe(false);
2424
});
25-
test('returns false for job id: "bad&job-name"', () => {
25+
test('returns false for job ID: "bad&job-name"', () => {
2626
expect(isTransformIdValid('bad&job-name')).toBe(false);
2727
});
2828
});

x-pack/legacy/plugins/transform/public/app/sections/create_transform/components/step_details/step_details_form.tsx

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@ export const StepDetailsForm: SFC<Props> = React.memo(({ overrides = {}, onChang
8888
} catch (e) {
8989
toastNotifications.addDanger(
9090
i18n.translate('xpack.transform.stepDetailsForm.errorGettingTransformList', {
91-
defaultMessage: 'An error occurred getting the existing transform Ids: {error}',
91+
defaultMessage: 'An error occurred getting the existing transform IDs: {error}',
9292
values: { error: JSON.stringify(e) },
9393
})
9494
);
@@ -181,7 +181,7 @@ export const StepDetailsForm: SFC<Props> = React.memo(({ overrides = {}, onChang
181181
<EuiForm>
182182
<EuiFormRow
183183
label={i18n.translate('xpack.transform.stepDetailsForm.transformIdLabel', {
184-
defaultMessage: 'Transform id',
184+
defaultMessage: 'Transform ID',
185185
})}
186186
isInvalid={(!transformIdEmpty && !transformIdValid) || transformIdExists}
187187
error={[
@@ -196,18 +196,18 @@ export const StepDetailsForm: SFC<Props> = React.memo(({ overrides = {}, onChang
196196
...(transformIdExists
197197
? [
198198
i18n.translate('xpack.transform.stepDetailsForm.transformIdExistsError', {
199-
defaultMessage: 'A transform with this id already exists.',
199+
defaultMessage: 'A transform with this ID already exists.',
200200
}),
201201
]
202202
: []),
203203
]}
204204
>
205205
<EuiFieldText
206-
placeholder="transform id"
206+
placeholder="transform ID"
207207
value={transformId}
208208
onChange={e => setTransformId(e.target.value)}
209209
aria-label={i18n.translate('xpack.transform.stepDetailsForm.transformIdInputAriaLabel', {
210-
defaultMessage: 'Choose a unique transform id.',
210+
defaultMessage: 'Choose a unique transform ID.',
211211
})}
212212
isInvalid={(!transformIdEmpty && !transformIdValid) || transformIdExists}
213213
/>

x-pack/legacy/plugins/transform/public/app/sections/create_transform/components/step_details/step_details_summary.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ export const StepDetailsSummary: SFC<StepDetailsExposedState> = React.memo(
3636
<Fragment>
3737
<EuiFormRow
3838
label={i18n.translate('xpack.transform.stepDetailsSummary.transformIdLabel', {
39-
defaultMessage: 'Transform id',
39+
defaultMessage: 'Transform ID',
4040
})}
4141
>
4242
<EuiFieldText defaultValue={transformId} disabled={true} />

x-pack/legacy/plugins/transform/public/app/sections/transform_management/components/transform_list/__snapshots__/expanded_row.test.tsx.snap

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

x-pack/legacy/plugins/transform/public/app/sections/transform_management/components/transform_list/action_start.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -60,14 +60,14 @@ export const StartAction: FC<StartActionProps> = ({ items, forceDisable }) => {
6060
startedTransformMessage = i18n.translate(
6161
'xpack.transform.transformList.startedTransformBulkToolTip',
6262
{
63-
defaultMessage: 'One or more selected transforms is already started.',
63+
defaultMessage: 'One or more transforms are already started.',
6464
}
6565
);
6666
completedBatchTransformMessage = i18n.translate(
6767
'xpack.transform.transformList.completeBatchTransformBulkActionToolTip',
6868
{
6969
defaultMessage:
70-
'One or more selected transforms is a completed batch transform and cannot be restarted.',
70+
'One or more transforms are completed batch transforms and cannot be restarted.',
7171
}
7272
);
7373
} else {

x-pack/legacy/plugins/transform/public/app/sections/transform_management/components/transform_list/action_stop.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ export const StopAction: FC<StopActionProps> = ({ items, forceDisable }) => {
3838
stoppedTransformMessage = i18n.translate(
3939
'xpack.transform.transformList.stoppedTransformBulkToolTip',
4040
{
41-
defaultMessage: 'One or more selected transforms is already stopped.',
41+
defaultMessage: 'One or more transforms are already stopped.',
4242
}
4343
);
4444
} else {

x-pack/legacy/plugins/transform/public/app/sections/transform_management/components/transform_list/expanded_row.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ export const ExpandedRow: SFC<Props> = ({ item }) => {
4343
const stateItems: Item[] = [];
4444
stateItems.push(
4545
{
46-
title: 'id',
46+
title: 'ID',
4747
description: item.id,
4848
},
4949
{

x-pack/legacy/plugins/transform/public/app/sections/transform_management/components/transform_list/transform_list.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -138,7 +138,7 @@ export const TransformList: FC<Props> = ({
138138

139139
if (c.type === 'term') {
140140
// filter term based clauses, e.g. bananas
141-
// match on id and description
141+
// match on ID and description
142142
// if the term has been negated, AND the matches
143143
if (bool === true) {
144144
ts = transforms.filter(

0 commit comments

Comments
 (0)