Skip to content

RED-150: Add table to view load on individual nodes on large-networks page#4

Merged
chrypnotoad merged 4 commits intodevfrom
RED-150
Jun 27, 2024
Merged

RED-150: Add table to view load on individual nodes on large-networks page#4
chrypnotoad merged 4 commits intodevfrom
RED-150

Conversation

@urnotsam
Copy link
Contributor

No description provided.

@linear
Copy link

linear bot commented Jun 25, 2024

RED-150 Monitor display table of queue data.

ISSUE SUMMARY:

View in monitor (large network page) where we have a 2d table of all nodes (refreshed per X seconds) where it displays # of txn in queue, length of txns, etc. So we can see which nodes are being backed up and by what.


ISSUE REPRO STEPS:

<HINT: Add steps to list as-needed. If interaction is complex, add screenshots or a Slack screen-capture video (just drag and drop)>

  1. <<TODO: Replace with repro step GOLD-82 feat: display queue details #1>>
  2. <<TODO: Replace with repro step Clean up large-network view fixes PM-70 #2>>
  3. Observe <<TODO: Describe unintended behavior.>>

EXPECTED RESULT:

<<TODO: Replace this with your expected results.>>


PULL REQUESTS:

<HINT: If your fix requires changes in multiple repos, add the following info per-repository.>

<<TODO: Enter Repository Name>>

Pull Request Link: <<TODO: Insert PR-LINK>>

GPT Review Link: <<TODO: Insert GPT-Review-Link>>

Jenkins Test Link: <<TODO: Insert Jenkins Test Job Link>>


ADDITIONAL INSTRUCTIONS:

<HINT: Add any additional instructions needed for the assignee. If you have specific requirements for how the task should be implemented or fixed, enter them or link them here.>

<<TODO: Insert additional instructions for assignee.>>

PudgyPug
PudgyPug previously approved these changes Jun 26, 2024
Comment on lines -6 to -8
<script src="axios.min.js"></script>
<script src="https://cdn.jsdelivr.net/npm/axios/dist/axios.min.js"></script>
<script src="https://unpkg.com/vis-network/standalone/umd/vis-network.min.js" type="text/javascript"></script>
<script src="vis-network-animated.js" type="text/javascript"></script>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did this need to be changed to use the cdn?

Copy link
Contributor

@chrypnotoad chrypnotoad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants