Skip to content

Commit

Permalink
fix: show Navie instead of walkthrough
Browse files Browse the repository at this point in the history
When sign in is required, open a Navie tab instead of showing the
walkthrough. Also, include a link to the walkthrough at the bottom of
the Documentation view.
  • Loading branch information
apotterri committed Oct 30, 2024
1 parent 22d4283 commit 922a302
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 2 deletions.
3 changes: 1 addition & 2 deletions src/services/signInManager.ts
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,6 @@ export default class SignInManager {
this.shouldShowSignIn()
);

if (!this.shouldShowSignIn())
vscode.commands.executeCommand('workbench.action.openWalkthrough', 'navie.walkthrough');
if (this.shouldShowSignIn()) vscode.commands.executeCommand('appmap.explain');
}
}
8 changes: 8 additions & 0 deletions src/tree/linkTreeDataProvider.ts
Original file line number Diff line number Diff line change
Expand Up @@ -83,6 +83,14 @@ export class LinkTreeDataProvider implements vscode.TreeDataProvider<vscode.Tree
return treeItem;
});

const walkthroughItem = new vscode.TreeItem('Walkthrough');
walkthroughItem.id = 'DOC_WALKTHROUGH' as string;
walkthroughItem.command = {
command: 'workbench.action.openWalkthrough',
arguments: ['appland.appmap#navie.walkthrough'],
} as vscode.Command;
items.push(walkthroughItem);

return Promise.resolve(items);
}

Expand Down

0 comments on commit 922a302

Please sign in to comment.