Skip to content
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: throw when csv produces non-unique cmdt record names #655

Merged
merged 1 commit into from
Nov 20, 2023

Conversation

mshanemc
Copy link
Contributor

forcedotcom/cli#2573
@W-14512810@

bug was coming from file cross-saves

QA:

repro issue with the CSV provided on 2573. You'll need to create the CMDT object/fields. Use this to get the records set up to match the csv.

sf cmdt generate object --label Table --type-name Table__mdt 
sf cmdt generate field --output-directory force-app/main/default/objects/Table__mdt --type text --name TelelinkInvoicingCompany__c
sf cmdt generate field --output-directory force-app/main/default/objects/Table__mdt --type Text --name TelelinkInvoicingCompany__c
sf cmdt generate field --output-directory force-app/main/default/objects/Table__mdt --type Text --name TelelinkCountryCodePick__c
sf cmdt generate field --output-directory force-app/main/default/objects/Table__mdt --type Text --name TelelinkCountryGroupCode__c
sf cmdt generate field --output-directory force-app/main/default/objects/Table__mdt --type Text --name TelelinkPostalCodeStart__c
sf cmdt generate field --output-directory force-app/main/default/objects/Table__mdt --type Text --name TelelinkPostalCodeEnd__c
sf cmdt generate field --output-directory force-app/main/default/objects/Table__mdt --type Text --name TelelinkVatSubjectCode__c
sf cmdt generate field --output-directory force-app/main/default/objects/Table__mdt --type Text --name TeleLinkCode__c

@mdonnalley mdonnalley merged commit 6ea936f into main Nov 20, 2023
12 checks passed
@mdonnalley mdonnalley deleted the sm/gh2573-bad-xml branch November 20, 2023 20:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants