Skip to content

Commit

Permalink
docs: message for custom fields wildcard (#1052)
Browse files Browse the repository at this point in the history
* docs: message for custom fields wildcard

* docs: fix message per PR discussion

* refactor: use common table header styler

* Update retrieve.start.md

---------

Co-authored-by: Juliet Shackell <63259011+jshackell-sfdc@users.noreply.github.com>
  • Loading branch information
mshanemc and jshackell-sfdc committed Jun 12, 2024
1 parent 5668eaa commit 07eb35d
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 6 deletions.
2 changes: 1 addition & 1 deletion messages/retrieve.start.md
Original file line number Diff line number Diff line change
Expand Up @@ -191,7 +191,7 @@ The retrieve target directory [%s] overlaps one of your package directories. Spe

# wantsToRetrieveCustomFields

Because you're retrieving one or more CustomFields, we're also retrieving the CustomObject to which it's associated.
Because you're retrieving one or more CustomFields that you didn't specify the name for, we're also retrieving the CustomObject to which they're associated.

# noSourceTracking

Expand Down
5 changes: 2 additions & 3 deletions src/commands/project/delete/source.ts
Original file line number Diff line number Diff line change
Expand Up @@ -33,10 +33,9 @@ import {
requiredOrgFlagWithDeprecations,
SfCommand,
} from '@salesforce/sf-plugins-core';
import ansis from 'ansis';
import { writeConflictTable } from '../../../utils/conflicts.js';
import { isNonDecomposedCustomLabel, isNonDecomposedCustomLabelsOrCustomLabel } from '../../../utils/metadataTypes.js';
import { getFileResponseSuccessProps } from '../../../utils/output.js';
import { getFileResponseSuccessProps, tableHeader } from '../../../utils/output.js';
import { API, DeleteSourceJson, isFileResponseDeleted, isSdrSuccess, isSourceComponent } from '../../../utils/types.js';
import { getPackageDirs, getSourceApiVersion } from '../../../utils/project.js';
import { resolveApi, validateTests } from '../../../utils/deploy.js';
Expand Down Expand Up @@ -187,7 +186,7 @@ export class Source extends SfCommand<DeleteSourceJson> {

if (!this.components.length) {
// if we didn't find any components to delete, let the user know and exit
this.styledHeader(ansis.blue('Deleted Source'));
this.styledHeader(tableHeader('Deleted Source'));
this.log('No results found');
return;
}
Expand Down
4 changes: 2 additions & 2 deletions src/formatters/deleteResultFormatter.ts
Original file line number Diff line number Diff line change
Expand Up @@ -7,14 +7,14 @@
import { Ux } from '@salesforce/sf-plugins-core';
import { DeployResult, FileResponse, FileResponseSuccess, RequestStatus } from '@salesforce/source-deploy-retrieve';
import { ensureArray } from '@salesforce/kit';
import ansis from 'ansis';
import { StandardColors } from '@salesforce/sf-plugins-core';
import { DeleteSourceJson, Formatter, TestLevel, isSdrSuccess } from '../utils/types.js';
import {
exitCodeAsNumber,
fileResponseSortFn,
getFileResponseSuccessProps,
makePathRelative,
tableHeader,
} from '../utils/output.js';
import { TestResultsFormatter } from '../formatters/testResultsFormatter.js';

Expand Down Expand Up @@ -80,7 +80,7 @@ export class DeleteResultFormatter extends TestResultsFormatter implements Forma
}

ux.log('');
ux.styledHeader(ansis.blue('Deleted Source'));
ux.styledHeader(tableHeader('Deleted Source'));
ux.table(
successes.map(getFileResponseSuccessProps),
{
Expand Down

0 comments on commit 07eb35d

Please sign in to comment.