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

[Gauge] Value range and limit indicators are broken #6593

Open
3 of 7 tasks
rukmini-bose opened this issue Apr 13, 2023 · 3 comments · May be fixed by #7684
Open
3 of 7 tasks

[Gauge] Value range and limit indicators are broken #6593

rukmini-bose opened this issue Apr 13, 2023 · 3 comments · May be fixed by #7684
Labels
bug:visual Visual problem. Not a functional issue help_wanted Help the Open MCT project! severity:medium type:bug

Comments

@rukmini-bose
Copy link
Contributor

rukmini-bose commented Apr 13, 2023

Summary

When a user manually enters high/low limits, the validity indicators (the red x or green check) that used to work, do not anymore. They stay showing invalid, regardless of whether the values a user inputs are valid or not. Additionally, when a user does enter an invalid limit, the OK button used to be disabled, disallowing users to save the Gauge until a valid value was inputted. This doesn't happen anymore– instead, users can save the object with invalid limits.

Notice in the picture below: The high limit of 90 IS valid if the maximum value is 100. However, the indicator shows it as invalid.
Additionally, the low limit of -20 IS NOT valid if the minimum value is 0. The indicator shows it as invalid as well. Now notice that the OK button is not disabled, and allows for a user to save the changes.
Screen Shot 2023-04-13 at 1 10 52 PM

Expected vs Current Behavior

Steps to Reproduce

  1. Navigate to/ create a gauge.
  2. Toggle off the ability to use Telemetry limits.
  3. Manually enter in your own limits. Test with invalid and valid limits. For limits to be valid, the low limit must be higher than the minimum value and the high limit must be lower than the maximum. For a limit to be invalid, the lower limit should be less than the minimum, and upper limit should be more.
  4. Observe that regardless of the limits being valid or not, the validity indicator stays invalid.
  5. Observe that when you enter an invalid limit, you are still able to press OK and save the object.

Environment

  • Open MCT Version: 2.2.1
  • Deployment Type: Testathon
  • OS:
  • Browser: Chrome

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 with no functional impact?
  • Does this block the execution of e2e tests?
  • Does this have an impact on Performance?

Additional Information

sinisam92 added a commit to sinisam92/openmct that referenced this issue Apr 20, 2023
@unlikelyzero unlikelyzero added bug:visual Visual problem. Not a functional issue severity:medium labels Jun 2, 2023
@unlikelyzero
Copy link
Collaborator

Note that this is a broken validation mechansim

@unlikelyzero unlikelyzero added the help_wanted Help the Open MCT project! label Jun 2, 2023
@TheFuturist32
Copy link

Hey all, I looked into these issues and was able to fix them. It's not allowing me to push to origin though, permission denied error.

@ozyx
Copy link
Member

ozyx commented Apr 10, 2024

Hey all, I looked into these issues and was able to fix them. It's not allowing me to push to origin though, permission denied error.

Awesome, thanks for looking into this. Please open a PR so we can review your changes. Also, you'll need to submit a CLA if you have not done so already. An approved CLA is needed before we can merge your code. We can still review in the meantime while we wait for that.

TheFuturist32 added a commit to TheFuturist32/openmct that referenced this issue Apr 12, 2024
fix: valid indicators not updating properly for Gauge plugin limits

refactor: enforce limitLow to be less than or equal to limitHigh
TheFuturist32 added a commit to TheFuturist32/openmct that referenced this issue Apr 12, 2024
fix: OK button not properly disabling when data is invalid

note: (implementation works around the broken validation mechanism)
@TheFuturist32 TheFuturist32 linked a pull request Apr 12, 2024 that will close this issue
14 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug:visual Visual problem. Not a functional issue help_wanted Help the Open MCT project! severity:medium type:bug
Projects
None yet
4 participants