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

@kobe1104 kobe1104 assigned kobe1104 and davetsay and unassigned kobe1104 Sep 10, 2021
@unlikelyzero unlikelyzero added this to Needs triage in Bug Tracker via automation Sep 22, 2021
@shefalijoshi shefalijoshi self-assigned this Jun 22, 2022
@unlikelyzero unlikelyzero added the needs:e2e Needs an e2e test label Jul 17, 2022
@unlikelyzero unlikelyzero added this to To triage in Improve Test Coverage via automation Jul 17, 2022
@unlikelyzero unlikelyzero modified the milestone: Sprint:2.0.7 Jul 17, 2022
@akhenry akhenry mentioned this issue Sep 19, 2022
15 tasks
Bug Tracker automation moved this from Needs triage to Done Sep 30, 2022
@shefalijoshi shefalijoshi added this to the Target:2.1.1 milestone Sep 30, 2022
@jvigliotta
Copy link
Contributor

Verified Fixed Testathon 9/30/22

@ozyx
Copy link
Member

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
Development

No branches or pull requests

6 participants