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

fix(app): enable about plate reader button when run is in progress #16803

Merged
merged 1 commit into from
Nov 13, 2024

Conversation

smb2268
Copy link
Contributor

@smb2268 smb2268 commented Nov 13, 2024

fix RQA-3561

Overview

Enable plate reader card "About Module" button while run is in progress

Test Plan and Hands on Testing

Check the module overflow button on a desktop devices page module card for a plate reader while the device has a run in progress. This should now be enabled

Changelog

Add isSettingDisabled prop to allow for some modules to have enabled overflow menu items even if a run is in progress. Make this false for plate reader "About Module" item so that it's always enabled

Review requests

Quick code check/test if you're in office

Risk assessment

Low

@smb2268 smb2268 requested a review from mjhuff November 13, 2024 22:02
@smb2268 smb2268 self-assigned this Nov 13, 2024
@smb2268 smb2268 requested a review from a team as a code owner November 13, 2024 22:02
@smb2268 smb2268 requested a review from vegano1 November 13, 2024 22:02
@@ -150,7 +150,7 @@ export const ModuleOverflowMenu = (
<Fragment key={`${index}_${String(module.moduleType)}`}>
<MenuItem
onClick={() => item.onClick(item.isSecondary)}
disabled={item.disabledReason || isDisabled}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This item.disabledReason field doesn't exist, but since item is typed as any here it wasn't causing an error

Copy link
Contributor

@mjhuff mjhuff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm - confirmed it works!

Screenshot 2024-11-13 at 5 30 12 PM

Copy link
Contributor

@vegano1 vegano1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@smb2268 smb2268 merged commit 20c98e6 into chore_release-8.2.0 Nov 13, 2024
36 checks passed
@smb2268 smb2268 deleted the app_plate-reader-disabled-setting branch November 13, 2024 22:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants