Skip to content

fix: add recipe ids W-11157722#118

Merged
jolley-sfdc merged 2 commits into
mainfrom
cjolley-add-recipeIds
May 18, 2022
Merged

fix: add recipe ids W-11157722#118
jolley-sfdc merged 2 commits into
mainfrom
cjolley-add-recipeIds

Conversation

@jolley-sfdc
Copy link
Copy Markdown
Contributor

What does this PR do?

Adds optional recipeids to the template:create and update command

What issues does this PR fix or reference?

@W-11157722@

@jolley-sfdc jolley-sfdc requested a review from a team as a code owner May 17, 2022 22:02
Comment thread README.md Outdated

EXAMPLE
$ sfdx analytics:template:create -f folderid
$ sfdx analytics:template:create -f folderid -r "05vxx0000004CAeAAM, 05vxx0000004C92AAE"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think these print out when you run help, I think typically i put something like versus an actual id

Copy link
Copy Markdown
Collaborator

@smithgp smithgp May 17, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe this portion of the README is actually auto-generated from the actual command source itself, so you don't need to add in the -r flag directly. Also, it get an example added here, you should instead update the examples class variable in the command .ts (e.g. create.ts). You can probably revert this file in the PR since it I think it's going to get overwritten on publish anyways.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah i missed that its in the readme file, this should go in create.ts, and just use "recipeid" versus an actual id

Comment thread messages/template.json Outdated
"assetVersionFlagLongDescription": "Version number for upgrading the template."
"assetVersionFlagLongDescription": "Version number for upgrading the template.",
"recipeidsFlagDescription": "comma-separated list of recipe IDs",
"recipeidsFlagLongDescription": "comma-separated list of recipe IDs to be includes with the template. "
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit:

Suggested change
"recipeidsFlagLongDescription": "comma-separated list of recipe IDs to be includes with the template. "
"recipeidsFlagLongDescription": "comma-separated list of recipe IDs to be included with the template. "

Comment thread README.md Outdated

EXAMPLE
$ sfdx analytics:template:create -f folderid
$ sfdx analytics:template:create -f folderid -r "05vxx0000004CAeAAM, 05vxx0000004C92AAE"
Copy link
Copy Markdown
Collaborator

@smithgp smithgp May 17, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe this portion of the README is actually auto-generated from the actual command source itself, so you don't need to add in the -r flag directly. Also, it get an example added here, you should instead update the examples class variable in the command .ts (e.g. create.ts). You can probably revert this file in the PR since it I think it's going to get overwritten on publish anyways.

Comment thread README.md Outdated

EXAMPLE
$ sfdx analytics:template:create -f folderid
$ sfdx analytics:template:create -f folderid -r "05vxx0000004CAeAAM, 05vxx0000004C92AAE"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah i missed that its in the readme file, this should go in create.ts, and just use "recipeid" versus an actual id

@jolley-sfdc jolley-sfdc merged commit cdfdd13 into main May 18, 2022
@jolley-sfdc jolley-sfdc deleted the cjolley-add-recipeIds branch May 18, 2022 14:42
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.

3 participants