manager icon indicating copy to clipboard operation
manager copied to clipboard

upcoming: [M3-9113] - Add Interface History table

Open coliu-akamai opened this issue 7 months ago โ€ข 1 comments

Description ๐Ÿ“

Adds the interface history dialog/table. Some background info: earlier in the year, the API team put this endpoint on hold for further reevaluation. They've since brought it back (see internal ticket comments for API's PRs), so we're adding this table back as well.

[!NOTE]
The API PR has not yet been merged. For now, I've added mocks via MSW crud to help with testing + will keep an eye out to merge this once the API's prs get merged (or wait until a release after)

Changes ๐Ÿ”„

  • Add dialog and table
  • Add react query / tests
  • Add mock data

Target release date ๐Ÿ—“๏ธ

depending on API's releasing of this endpoint, keeping an eye out

Preview ๐Ÿ“ท

button dialog + table
image image

How to test ๐Ÿงช

Prerequisites

For now, use MSW crud and create a Linode using new interfaces (once the API pr gets merged, we'll be able to test with live data via devenv)

Verification steps

  • confirm history table dialog opens
  • confirm it matches the screenshot in the ticket
    • to test pagination, change the number of history objects returned in handlers/linodes.ts
  • confirm tests pass
Author Checklists

As an Author, to speed up the review process, I considered ๐Ÿค”

๐Ÿ‘€ Doing a self review โ” Our contribution guidelines ๐Ÿค Splitting feature into small PRs โž• Adding a changeset ๐Ÿงช Providing/improving test coverage ๐Ÿ” Removing all sensitive information from the code and PR description ๐Ÿšฉ Using a feature flag to protect the release ๐Ÿ‘ฃ Providing comprehensive reproduction steps ๐Ÿ“‘ Providing or updating our documentation ๐Ÿ•› Scheduling a pair reviewing session ๐Ÿ“ฑ Providing mobile support โ™ฟ Providing accessibility support


  • [ ] I have read and considered all applicable items listed above.

As an Author, before moving this PR from Draft to Open, I confirmed โœ…

  • [ ] All unit tests are passing
  • [ ] TypeScript compilation succeeded without errors
  • [ ] Code passes all linting rules

coliu-akamai avatar Jun 03 '25 16:06 coliu-akamai

This PR is stale because it has been open 15 days with no activity. Please attend to this PR or it will be closed in 5 days

github-actions[bot] avatar Sep 10 '25 00:09 github-actions[bot]

I feature flagged the Interface History table so we can go ahead and merge it to keep it from going stale.

The feature works in lower environments, we just need to toggle the flag once this is available in production.

Screenshot 2025-09-16 at 11 37 11โ€ฏAM

bnussman-akamai avatar Sep 16 '25 15:09 bnussman-akamai

Cloud Manager UI test results

:tada: 809 passing tests on test run #28 โ†—๏ธŽ

:x: Failing:white_check_mark: Passing:arrow_right_hook: Skipped:clock1: Duration
0 Failing809 Passing9 Skipped44m 45s

linode-gh-bot avatar Sep 17 '25 15:09 linode-gh-bot