Skip to content
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

[Tabs View] Making requests when switching tabs while eager load is true #4206

Closed
1 of 5 tasks
kobe1104 opened this issue Sep 10, 2021 · 2 comments
Closed
1 of 5 tasks

Comments

@kobe1104
Copy link
Contributor

kobe1104 commented Sep 10, 2021

The tabs view is making new requests when switching tabs while eager load is true.
It should not make new request unless clock mode is changed while eager load is true since they are already loaded.

Summary

Expected vs Current Behavior

Expected: No new requests should be made while switching tabs.
Current: Making new requests when switching tabs.

Impact Check List

  • Data loss or misrepresented data?
  • Regression? Did this used to work or has it always been broken?
  • Is there a workaround available?
  • Does this impact a critical component?
  • Is this just a visual bug?

Steps to Reproduce

  1. Create a Tabs view 'eager load' is true and populate it with at least 2 telemetry sources.
  2. Open network tab in browser dev tools and clear all the requests.
  3. Switch to a different tab and observe new requests for all sources have been made.

Environment

  • Open MCT Version: Version: 1.7.8-SNAPSHOT
    Build Date: Thu Sep 09 2021 12:20:09 GMT-0700 (Pacific Daylight Time)
  • Deployment Type:
  • OS: IOS 10.15.7
  • Browser: Chrome 92.0.4515.159

Additional Information

@jvigliotta
Copy link
Contributor

Verified Fixed Testathon 9/30/22

@ozyx
Copy link
Contributor

ozyx commented Sep 30, 2022

Verified Fixed Testathon 09/30/22

Did not observe any network requests while switching between tabs

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

No branches or pull requests

6 participants