-
Notifications
You must be signed in to change notification settings - Fork 6
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: fix broken link in results-manager template #1313
docs: fix broken link in results-manager template #1313
Conversation
Thanks for your contribution @mrrajamanickam-coveo !
|
Dependency Review✅ No vulnerabilities or license issues found.Scanned Manifest Files |
Pull Request Report PR Title ✅ Title follows the conventional commit spec. |
packages/ui/atomic/template/src/components/results-manager/results-manager.tsx.hbs
Show resolved
Hide resolved
Don't think the changes in this PR updating a doc string should cause the CI error
Shall I proceed to merge @louis-bompart ? |
Lgtm. |
Not sure what updating snapshots means in the context of CI. Is it jest snapshots? Tried to follow the instruction "run Will merge this in the interest of getting the long open PR closed. |
Proposed changes