Skip to content
This repository has been archived by the owner on Mar 25, 2021. It is now read-only.

Implement model & controller for Topics (listing) page #124

Closed
pmuir opened this issue Nov 19, 2020 · 2 comments · May be fixed by #136
Closed

Implement model & controller for Topics (listing) page #124

pmuir opened this issue Nov 19, 2020 · 2 comments · May be fixed by #136
Assignees
Labels

Comments

@pmuir
Copy link
Contributor

pmuir commented Nov 19, 2020

Implement the model & controller for the Topics (listing) page. #70 contains the designs. See also #123 and #122 for the view implementations.

@matthew-chirgwin
Copy link
Member

For the first stage, I am hoping this will be a simple model hook, calling a useQuery to get the data/deal with loading/errors etc.

I expect while this will be quite boilerplate like for now, other functions etc will follow/be added to this model

@ajborley ajborley self-assigned this Nov 30, 2020
ajborley added a commit to ajborley/strimzi-ui that referenced this issue Dec 2, 2020
 - This commit adds a new `useTopics` model hook, which makes GraphQL
requests to the server to administer Kafka topics.
 - The hook returns a set of sub-hooks, with just the `useGetTopics`
sub-hook implemented for now which returns the list of topics (including
their name, number of partitions and number of replicas).

Contributes to: strimzi#124

Signed-off-by: Andrew Borley <[email protected]>
ajborley added a commit to ajborley/strimzi-ui that referenced this issue Dec 2, 2020
 - This commit adds a new `useTopics` model hook, which makes GraphQL
requests to the server to administer Kafka topics.
 - The hook returns a set of sub-hooks, with just the `useGetTopics`
sub-hook implemented for now. `useGetTopics` returns the list of topics
(including name, number of partitions and number of replicas).

Contributes to: strimzi#124

Signed-off-by: Andrew Borley <[email protected]>
ajborley added a commit to ajborley/strimzi-ui that referenced this issue Dec 2, 2020
 - This commit adds a new `useTopics` model hook, which makes GraphQL
requests to the server to administer Kafka topics.
 - The hook returns a set of sub-hooks, with just the `useGetTopics`
sub-hook implemented for now. `useGetTopics` returns the list of topics
(including name, number of partitions and number of replicas).

Contributes to: strimzi#124

Signed-off-by: Andrew Borley <[email protected]>
ajborley added a commit to ajborley/strimzi-ui that referenced this issue Dec 4, 2020
 - This commit adds a new `useTopics` model hook, which makes GraphQL
requests to the server to administer Kafka topics.
 - The hook returns a set of sub-hooks, with just the `useGetTopics`
sub-hook implemented for now. `useGetTopics` returns the list of topics
(including name, number of partitions and number of replicas).

Contributes to: strimzi#124

Signed-off-by: Andrew Borley <[email protected]>
ajborley added a commit to ajborley/strimzi-ui that referenced this issue Dec 4, 2020
 - Addresses initial review comments.

Contributes to: strimzi#124

Signed-off-by: Andrew Borley <[email protected]>
ajborley added a commit to ajborley/strimzi-ui that referenced this issue Dec 7, 2020
 - Based on review comments from @matthew-chirgwin

Contributes to: strimzi#124

Signed-off-by: Andrew Borley <[email protected]>
pmuir pushed a commit to pmuir/strimzi-ui that referenced this issue Dec 9, 2020
 - This commit adds a new `useTopics` model hook, which makes GraphQL
requests to the server to administer Kafka topics.
 - The hook returns a set of sub-hooks, with just the `useGetTopics`
sub-hook implemented for now. `useGetTopics` returns the list of topics
(including name, number of partitions and number of replicas).

Contributes to: strimzi#124

Signed-off-by: Andrew Borley <[email protected]>
pmuir pushed a commit to pmuir/strimzi-ui that referenced this issue Dec 9, 2020
 - Addresses initial review comments.

Contributes to: strimzi#124

Signed-off-by: Andrew Borley <[email protected]>
pmuir pushed a commit to pmuir/strimzi-ui that referenced this issue Dec 9, 2020
 - Based on review comments from @matthew-chirgwin

Contributes to: strimzi#124

Signed-off-by: Andrew Borley <[email protected]>
@github-actions
Copy link

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 7 days

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants