-
Notifications
You must be signed in to change notification settings - Fork 324
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
Enhancement: Added copilot alias to chatbot commands #6386
base: main
Are you sure you want to change the base?
Conversation
Hi @ktskumar, could you fix the error in the docs? Seems like you'll have to update the navigation nodes to the pages.
|
Hi @milanholemans, I have updated the navigation code. Pls review it |
Seems like there are still broken links in the release notes, could you have a look at it? |
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.
Nice start, let's do a couple of improvements before we merge it.
m365 pp copilot get [options] | ||
``` | ||
|
||
## alias |
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.
Let's start with an uppercase letter.
m365 pp copilot list [options] | ||
``` | ||
|
||
## alias |
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.
Let's start with an uppercase letter.
@@ -79,26 +85,26 @@ m365 pp chatbot list --environmentName "Default-d87a7535-dd31-4437-bfe1-95340acd | |||
```text | |||
name botid publishedOn createdOn botModifiedOn |
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.
Seems like the headers are a bit out of sync. Can you fix that as well?
m365 pp copilot remove [options] | ||
``` | ||
|
||
## alias |
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.
Let's start with an uppercase letter.
@@ -89,21 +93,21 @@ class PpChatbotGetCommand extends PowerPlatformCommand { | |||
|
|||
public async commandAction(logger: Logger, args: CommandArgs): Promise<void> { | |||
if (this.verbose) { |
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.
As first thing to do, let's log a deprecation message to notify our users that the old name will go away. You can use the this.showDeprecationWarning(...)
for this.
@@ -55,7 +59,7 @@ class PpChatbotListCommand extends PowerPlatformCommand { | |||
|
|||
public async commandAction(logger: Logger, args: CommandArgs): Promise<void> { | |||
if (this.verbose) { |
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.
Let's log a deprecation message here as well.
|
||
it('defines alias', () => { | ||
const alias = command.alias(); | ||
assert.notStrictEqual(typeof alias, 'undefined'); |
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.
If we check for an alias, let's use an exact match on the old name.
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.
Let's also add a docs redirect from the old chatbot URLs to the new Copilot URLs:
cli-microsoft365/docs/docusaurus.config.ts
Lines 49 to 58 in 67e608f
'client-redirects', | |
{ | |
createRedirects(routePath) { | |
if (routePath.includes('/entra')) { | |
return [routePath.replace('/entra', '/aad')]; | |
} | |
return []; | |
} | |
} satisfies ClientRedirectsOptions |
Closes #6261
Enhancement:
Updated below commands,
[☑] Rename
pp chatbot get
topp copilot get
[☑] Add alias
pp chatbot get
to renamed command[☑] Rename
pp chatbot list
topp copilot list
[☑] Add alias
pp chatbot list
to renamed command[☑] Rename
pp chatbot remove
topp copilot remove
[☑] Add alias
pp chatbot remove
to renamed command