Skip to content

Added monitor test screen#47

Open
adedamola-sode wants to merge 4 commits intomainfrom
monotor-test-screen
Open

Added monitor test screen#47
adedamola-sode wants to merge 4 commits intomainfrom
monotor-test-screen

Conversation

@adedamola-sode
Copy link
Copy Markdown
Collaborator

Added the monitor test/sample screen.

Copy link
Copy Markdown
Collaborator

@GregJHarris GregJHarris left a comment

Choose a reason for hiding this comment

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

You have got the widgets in place, but it would be nice to hook them up to some SIM pvs so that we can see them in action,

Copy link
Copy Markdown
Collaborator

@GregJHarris GregJHarris left a comment

Choose a reason for hiding this comment

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

Thanks Ade. There are a couple of changes required to get this working with cs-web-lib, I've highlighted these inline. Please could you merger in main, set the version number to 0.1.11 and run npm install. Thanks.

Comment thread public/BOBs/test/monitor.bob Outdated
Comment thread public/BOBs/test/monitor.bob Outdated
Comment thread public/BOBs/test/monitor.bob Outdated
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.

2 participants