Skip to content

Commit 54005ee

Browse files
committed
fix(providers): link /providers/quota to tracking issue #132
The follow-up GitHub issue for the real per-provider quota integration has been filed (#132). The quota view now links to it and the TODO comment references it instead of the placeholder text.
1 parent b1eb991 commit 54005ee

1 file changed

Lines changed: 15 additions & 6 deletions

File tree

‎apps/rook/dashboard/src/views/ProvidersQuotaView.vue‎

Lines changed: 15 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515
*/
1616
import { onMounted } from 'vue'
1717
import { useI18n } from 'vue-i18n'
18-
import { Fuel, Info } from '@lucide/vue'
18+
import { Fuel, Info, ExternalLink } from '@lucide/vue'
1919
import { Alert, AlertDescription, AlertTitle } from '@/components/ui/alert'
2020
import { Badge } from '@/components/ui/badge'
2121
import { Table, TableBody, TableCell, TableHead, TableHeader, TableRow } from '@/components/ui/table'
@@ -34,9 +34,11 @@ onMounted(() => {
3434
fetchModels()
3535
})
3636
37-
// TODO: replace with the real implementation once the backend exposes
38-
// a quota endpoint. Track the follow-up work in the GitHub issue
39-
// linked from the spec/design doc.
37+
// TODO(issue #132): replace with the real per-provider quota data once the
38+
// backend exposes a QuotaPort. See:
39+
// https://github.com/dallay/cortex/issues/132
40+
// Spec: openspec/changes/providers-ui-3-screen-refactor/specs/providers-ui/spec.md (REQ-8)
41+
// Design: openspec/changes/providers-ui-3-screen-refactor/design.md (D7)
4042
</script>
4143

4244
<template>
@@ -56,8 +58,15 @@ onMounted(() => {
5658
<AlertTitle>{{ t('providers.quota.comingSoon') }}</AlertTitle>
5759
<AlertDescription>
5860
{{ t('providers.quota.banner') }}
59-
<!-- TODO: link to the implementation tracking issue once filed.
60-
Do not hardcode a GitHub URL here until the issue exists. -->
61+
<a
62+
href="https://github.com/dallay/cortex/issues/132"
63+
target="_blank"
64+
rel="noopener noreferrer"
65+
class="underline ml-1 inline-flex items-center gap-1"
66+
>
67+
{{ t('providers.quota.followUpIssue') }}
68+
<ExternalLink class="h-3 w-3" />
69+
</a>
6170
</AlertDescription>
6271
</Alert>
6372

0 commit comments

Comments
 (0)