-
Notifications
You must be signed in to change notification settings - Fork 99
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
feat: mdTypes from core export #1398
Conversation
QA Notes❌ : sf alias set registryBuilder=<my org>
# changed metadata registry entries, expecting next command to revert changes
➜ yarn update-registry-org CustomField ListView customnotificationtype
yarn run v1.22.22
$ npx ts-node scripts/update-registry/updateRegistryFromOrg.ts CustomField ListView customnotificationtype
Will get describe for these types:
- CustomField
- ListView
- customnotificationtype
[]
✨ Done in 2.85s.
# no changes to file ❌ : from the ➜ yarn update-registry-org
yarn run v1.22.22
$ npx ts-node scripts/update-registry/updateRegistryFromOrg.ts
/Users/william.ruemmele/projects/oss/source-deploy-retrieve/scripts/update-registry/updateRegistryFromOrg.ts:38
})();
^
Error: Please provide the types to update the registry with. Ex: `yarn update-registry-org CustomObject CustomField`
at /Users/william.ruemmele/projects/oss/source-deploy-retrieve/scripts/update-registry/updateRegistryFromOrg.ts:29:11 |
fyi, customnotificationtype will work with the next change. CustomField and ListView aren't in the describe because they're not top-level types but are children of CustomObject. |
Ooh, good docs catch. I meant to disable that option for the |
figured out the CF/LV wouldn't work, and then happened to choose another type that wouldn't work... unlucky on my part. I'll try again |
QA Notes✅ : changed ✅ : |
What does this PR do?
given that the file has been copied from the core test branch to SDR, add them to the registry
What issues does this PR fix or reference?
@W-16495885@