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
fix(nms): Initially load count of subscribers correctly #12383
Conversation
Thanks for opening a PR! 💯
Howto
More infoPlease take a moment to read through the Magma project's
If this is your first Magma PR, also consider reading
|
@andreilee Is there a good way to also get automatic updates in? I think it would be possible to do that in |
3a53d11
to
92acf63
Compare
92acf63
to
51113e7
Compare
51113e7
to
66c43a2
Compare
Rebased. |
66c43a2
to
7f585d6
Compare
@@ -43,6 +43,7 @@ export type SubscriberContextType = { | |||
forbiddenNetworkTypes: {[string]: core_network_types}, | |||
metrics?: {[string]: Metrics}, | |||
gwSubscriberMap: {[gateway_id]: Array<subscriber_id>}, | |||
totalCount: number, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: update the comments above
Fixes a bug where the number of subscribers shown in the KPI panel depends on the pagination of the subscriber table. Does not handle changes to the number without reload. Signed-off-by: Sebastian Thomas <sebastian.thomas@tngtech.com>
7f585d6
to
ba1759f
Compare
Fixes a bug where the number of subscribers shown in the KPI panel depends on the pagination of the subscriber table. Does not handle changes to the number without reload. Signed-off-by: Sebastian Thomas <sebastian.thomas@tngtech.com>
Summary
Fixes a bug where the number of subscribers shown in the KPI panel depends on the pagination of the subscriber table.
Does not handle changes to the number without reloading NMS.
Fixes #11929.
Thanks @Siddharthlende for investigating!
Done in pairing with @thmsschmitt.
Test Plan
Set up NMS locally with 98 subscribers.
Additional Information