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

docs: Correct cmd file path for manage configuration in README.md #2

Merged
merged 1 commit into from
Dec 28, 2023

Conversation

rmbrntt
Copy link
Contributor

@rmbrntt rmbrntt commented Dec 28, 2023

Type of change (place an x in the [ ] that applies)

  • New sample
  • New feature
  • Bug fix
  • Documentation

Summary

README has the command slack trigger create --trigger-def triggers/manage_configuration_trigger.ts which doesn't exist so it throws an error
image

Actual file path in repo is triggers/manage_configuration_shortcut_trigger.ts

PR updates the command listed in the README to correct location.

Requirements (place an x in each [ ] that applies)

  • I’ve checked my submission against the Samples Checklist to ensure it complies with all standards
  • I have ensured the changes I am contributing align with existing patterns and have tested and linted my code
  • I've read and agree to the Code of Conduct

Copy link

Thanks for the contribution! Before we can merge this, we need @rmbrntt to sign the Salesforce Inc. Contributor License Agreement.

Copy link

@hello-ashleyintech hello-ashleyintech left a comment

Choose a reason for hiding this comment

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

🙌 LGTM! Thanks for updating this 😄

@hello-ashleyintech hello-ashleyintech merged commit 97dfade into slack-samples:main Dec 28, 2023
2 checks passed
@hello-ashleyintech hello-ashleyintech added the documentation Improvements or additions to documentation label Dec 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla:signed documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants