-
Notifications
You must be signed in to change notification settings - Fork 102
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
fix: force fullName to CustomLabels #427
Conversation
dfde4a3
to
879129f
Compare
src/resolve/sourceComponent.ts
Outdated
@@ -218,6 +218,9 @@ export class SourceComponent implements MetadataComponent { | |||
} | |||
|
|||
get fullName(): string { | |||
if (this.type.ignoreTypeName) { |
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.
NIT: the naming makes this line confusing, if ignoreTypeName
is true, we return this.type.name
which I wouldn't expect to happen if I set ignoreTypeName=true
. From the JSDoc, what if we change ignoreTypeName
to ignoreParsedFileName
But I confirmed the change works,
- deploy
CustomLabels.labels-meta.xml
- deploys
Labels.labels-meta.xml
- deploys
force-app/main/default/myLabels/Labels.labels-meta.xml
- retrieves into
myLabels/Labels...
- retrieves into
labels/CustomLabels..
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.
I test this on MPD deploy-retrieve scenarios and did source:convert
with it, too.
* fix: force fullName to CustomLabels * chore: update prop name * fix: backward support for toolbelt suffix differences * Revert "fix: backward support for toolbelt suffix differences" This reverts commit 9cf9505. Co-authored-by: mshanemc <[email protected]> Co-authored-by: Willie Ruemmele <[email protected]>
* fix: export all public TS types and interfaces (#423) * fix: making exports in the right format @W-9727845@ * chore: removing yarn changes * fix: xml nodes might not be arrays (#424) Co-authored-by: Willie Ruemmele <[email protected]> * fix: throw an error for unexpected child types (#426) * fix: throw an error for unexpected child types * fix: throw better error when transforming invalid child * fix: ensure valid children during retrieve Co-authored-by: Willie Ruemmele <[email protected]> * fix: force fullName to CustomLabels (#427) * fix: force fullName to CustomLabels * chore: update prop name * fix: backward support for toolbelt suffix differences * Revert "fix: backward support for toolbelt suffix differences" This reverts commit 9cf9505. Co-authored-by: mshanemc <[email protected]> Co-authored-by: Willie Ruemmele <[email protected]> * fix: forceIgnore does not work for `SFDX: Retrieve Source in Manifest from Org` in vscode extension (#413) * fix: forceIgnore does not work with retrieve manifest * fix: matchingContent subfolder * Wr/content type of undefined (#429) * chore: initial pass on preventing improperly structured static resources * chore: added error message Co-authored-by: Shane McLaughlin <[email protected]> * fix: support toolbelt suffixes (#428) * fix: force fullName to CustomLabels * chore: update prop name * fix: backward support for toolbelt suffix differences * Revert "fix: backward support for toolbelt suffix differences" This reverts commit 9cf9505. * fix: support toolbelt suffixes * feat: support legacy suffix conversion Co-authored-by: Mike Donnalley <[email protected]> Co-authored-by: Willie Ruemmele <[email protected]> Co-authored-by: vamsimundra <[email protected]> Co-authored-by: Shane McLaughlin <[email protected]> Co-authored-by: Willie Ruemmele <[email protected]> Co-authored-by: Steve Hetzel <[email protected]> Co-authored-by: Mike Donnalley <[email protected]> Co-authored-by: Violet Yao <[email protected]>
What does this PR do?
Adds
ignoreParsedFullName
option to registry. Whentrue
this will ignore the name that was parsed from the file path and use the metadata type instead. Doing so fixes deploy and retrieve for users who have CustomLabels files not namedCustomLabels.labels-meta.xml
, e.g.MyLabels.labels-meta.xml
What issues does this PR fix or reference?
forcedotcom/cli#1133,
@W-9720198@