hendrik/lens
1
0
Fork 0
mirror of https://github.com/lensapp/lens.git synced 2025-05-20 05:10:56 +00:00
Code Issues Projects Releases Wiki Activity

remove deprecated routePath before GA

Browse Source 
Signed-off-by: Sebastian Malton <sebastian@malton.name>
This commit is contained in:
Sebastian Malton 2020-11-24 15:34:53 -05:00
parent 4e6b8ee10e
commit 97e99614c8
6 changed files with 7 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
docs/extensions/capabilities/common-capabilities.md
View File

@ -101,7 +101,6 @@ export default class ExampleRendererExtension extends LensRendererExtension {
globalPages = [
{
id: "example",
routePath: "/example",
components: {
Page: ExamplePage,
}
@ -156,7 +155,7 @@ import { ExampleIcon, ExamplePage } from "./src/page"
export default class ExampleExtension extends LensRendererExtension {
clusterPages = [
{
routePath: "/extension-example", // optional
id: "extension-example", // optional
exact: true, // optional
components: {
Page: () => <ExamplePage extension={this}/>,

4
docs/extensions/get-started/anatomy.md
View File

@ -85,7 +85,7 @@ import React from "react"
export default class ExampleExtension extends LensRendererExtension {
clusterPages = [
{
routePath: "/extension-example",
id: "extension-example",
components: {
Page: () => <ExamplePage extension={this}/>,
}
@ -94,4 +94,4 @@ export default class ExampleExtension extends LensRendererExtension {
}
```
The Hello World sample extension uses the `Cluster Page` capability, which is just one of the Lens extension API's capabilities. The [Common Capabilities](../capabilities/common-capabilities.md) page will help you home in on the right capabilities to use with your own extensions.
The Hello World sample extension uses the `Cluster Page` capability, which is just one of the Lens extension API's capabilities. The [Common Capabilities](../capabilities/common-capabilities.md) page will help you home in on the right capabilities to use with your own extensions.

2
extensions/example-extension/package.json
View File

@ -1,5 +1,5 @@
{
"name": "extension-example",
"name": "example-extension",
"version": "1.0.0",
"description": "Example extension",
"main": "dist/main.js",

1
extensions/example-extension/renderer.tsx
View File

@ -6,7 +6,6 @@ export default class ExampleExtension extends LensRendererExtension {
clusterPages = [
{
id: "example",
routePath: "/extension-example",
title: "Example Extension",
components: {
Page: () => <ExamplePage extension={this}/>,

5
src/extensions/extension-discovery.ts
View File

@ -91,7 +91,7 @@ export class ExtensionDiscovery {
init() {
this.watchExtensions();
}
/**
* Watches for added/removed local extensions.
* Dependencies are installed automatically after an extension folder is copied.
@ -303,7 +303,6 @@ export class ExtensionDiscovery {
}
const extension = await this.loadExtensionFromPath(absPath);
if (extension) {
extensions.push(extension);
}
@ -326,4 +325,4 @@ export class ExtensionDiscovery {
}
}
export const extensionDiscovery = new ExtensionDiscovery();
export const extensionDiscovery = new ExtensionDiscovery();

7
src/extensions/registries/page-registry.ts
View File

@ -16,11 +16,6 @@ export interface PageRegistration {
* When not provided, first registered page without "id" would be used for page-menus without target.pageId for same extension
*/
id?: string;
/**
* Alias to page ID which assume to be used as path with possible :param placeholders
* @deprecated
*/
routePath?: string;
/**
* Strict route matching to provided page-id, read also: https://reactrouter.com/web/api/NavLink/exact-bool
* In case when more than one page registered at same extension "pageId" is required to identify different pages,
@ -69,7 +64,7 @@ export class PageRegistry extends BaseRegistry<RegisteredPage> {
registeredPages = itemArray.map(page => ({
...page,
extensionId: ext.name,
routePath: getExtensionPageUrl({ extensionId: ext.name, pageId: page.id ?? page.routePath }),
routePath: getExtensionPageUrl({ extensionId: ext.name, pageId: page.id }),
}));
} catch (err) {
logger.error(`[EXTENSION]: page-registration failed`, {