refactor: [M3-8908] - Optimize Events Polling following changes from incident

[bnussman-akamai]

Description 📝

• Cleans up events caching logic following change: [M3-8450] - Lower Events historical data fetching to 7 days  #10899 🧹

Changes 🔄

• Removes the useInitialEventsQuery hook and moves all logic into useEventsInfiniteQuery, similar to the implementation before change: [M3-8450] - Lower Events historical data fetching to 7 days  #10899
• Adds logic to useEventsInfiniteQuery that ensures the first fetch always limited to a 7 day window 📆

Preview 📷

Note

No UI changes expected

How to test 🧪

• Test the Events landing page in general
  • Test on an account with 0 events
  • Test infinite scrolling
  • Test on an account with 0 events within the last 7 days, but events past that 7 day period
• Test the activity feed on a Linode details page
• Verify events polling works as expected
  • For example, if you shutdown a Linode, you should see the percentage for that every update every few seconds
• Verify events get marked as seen as expected

As an Author, I have 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

[abailly-akamai]

Solid PR - did not notice any regression with any of the eventing scenarios I tried:

• event polling ✅
• events page (with and without events) ✅
• events page infinite scrolling ✅
• linode activity ✅

@bnussman-akamai only thing is you'll need is a changeset

[github-actions]

Coverage Report: ✅
Base Coverage: 86.88%
Current Coverage: 86.88%

[linode-gh-bot]

Cloud Manager UI test results

🎉 453 passing tests on test run #6 ↗︎

❌ Failing	✅ Passing	↪️ Skipped	🕐 Duration
0 Failing	453 Passing	2 Skipped	83m 41s

[mjac0bs]

Verified:

• Events landing page in general
  • Mocking 0 events displays no results and the 'You don't have any events on your account' copy
  • Mocking 0 events in the last 7 days but older events past 7 days displays those older events
  • Events infinitely scroll
• The activity feed on a Linode details page shows recent and past 7 days events for that Linode
• Polling works - the progress bar for in-progress events updates every few seconds that polling happens
• Events are marked as seen when the notification center is viewed

[cypress]

Cloud Manager E2E    Run #6850

Run Properties:   Failed #6850  •  426be755c6: refactor: [M3-8908] - Optimize Events Polling following changes from incident (#...

Project	Cloud Manager E2E
Branch Review	develop
Run status	Failed #6850
Run duration	26m 39s
Commit	426be755c6: refactor: [M3-8908] - Optimize Events Polling following changes from incident (#...
Committer	Banks Nussman
View all properties for this run ↗︎

---

Test results
Failures	1
Flaky	0
Pending	2
Skipped	0
Passing	452
View all changes introduced in this branch ↗︎

---

Tests for review

  cypress/e2e/core/stackscripts/update-stackscripts.spec.ts • 1 failed test

View Output Video

Test		Artifacts
Update stackscripts > updates a StackScript		Screenshots Video