-
Notifications
You must be signed in to change notification settings - Fork 7
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
Implement command palette #353
Conversation
Visit the preview URL for this PR (updated for commit 44b1c11): https://pluralsh-console--pr353-kbar-tdd9xt8s.web.app (expires Fri, 19 May 2023 13:08:22 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: dd1ffa0705acc6ef7d6db370e6bd6fc390e945ce |
74b473c
to
44b1c11
Compare
assets/package.json
Outdated
@@ -67,6 +67,7 @@ | |||
"js-file-download": "0.4.12", | |||
"js-yaml": "4.1.0", | |||
"jsonpath": "1.1.1", | |||
"kbar": "^0.1.0-beta.40", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you change to 0.1.0-beta.40
?
} | ||
|
||
function buildActions(applications, nav) { | ||
console.log(applications) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you remove it?
// icon: <BuildIcon />, | ||
// section: 'Cluster', | ||
// perform: () => navigate('/builds'), | ||
// }), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should it stay commented out?
d05fef3
to
e87f566
Compare
Currently supports basic app nav, but will also integrate the ai endpoint functionality too
2b26350
to
d100d22
Compare
Summary
Currently supports basic app nav, but will also integrate the ai endpoint functionality too
Test Plan
local + unit tests
Checklist