-
Notifications
You must be signed in to change notification settings - Fork 401
refactor: [M3-9510] - Clean up main search implementation #11819
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
Changes from 24 commits
aac2a0c
87690d6
1b54973
c702494
56fdbc3
a3dfce0
b94656d
be9b93d
f18a145
db8b214
53236a9
5ce8b2c
0f72f14
682083a
ea20290
59c611b
aa5e67a
a45094c
2475c9d
7f40f25
36d57a6
96cca3d
dc611ba
1b68d0d
366689b
268bd3b
81d28e5
8c336b7
ca9d237
b863fbe
882356c
0670da8
da18128
e4f59e1
1ad1e52
7c6f873
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| --- | ||
| "@linode/manager": Changed | ||
| --- | ||
|
|
||
| Updates main search to use new API search implimentation for large accounts ([#11819](https://github.com/linode/manager/pull/11819)) | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| --- | ||
| "@linode/manager": Tech Stories | ||
| --- | ||
|
|
||
| Updates main search to not depend on recompose ([#11819](https://github.com/linode/manager/pull/11819)) | ||
|
bnussman-akamai marked this conversation as resolved.
Outdated
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -17,7 +17,6 @@ import type { | |
| KubernetesTieredVersion, | ||
| KubernetesVersion, | ||
| } from '@linode/api-v4/lib/kubernetes'; | ||
| import type { Region } from '@linode/api-v4/lib/regions'; | ||
| import type { ExtendedType } from 'src/utilities/extendType'; | ||
|
|
||
| interface ClusterData { | ||
|
|
@@ -52,14 +51,10 @@ export const getTotalClusterMemoryCPUAndStorage = ( | |
| ); | ||
| }; | ||
|
|
||
| export const getDescriptionForCluster = ( | ||
| cluster: KubernetesCluster, | ||
| regions: Region[] | ||
| ) => { | ||
| const region = regions.find((r) => r.id === cluster.region); | ||
| export const getDescriptionForCluster = (cluster: KubernetesCluster) => { | ||
| const description: string[] = [ | ||
| `Kubernetes ${cluster.k8s_version}`, | ||
| region?.label ?? cluster.region, | ||
|
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The one trade-off this PR makes is that our " |
||
| cluster.region, | ||
| ]; | ||
|
|
||
| if (cluster.control_plane.high_availability) { | ||
|
|
||
This file was deleted.
This file was deleted.
This file was deleted.
Uh oh!
There was an error while loading. Please reload this page.