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

Add agent status and enabled state to page summary #177

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

humberto-garza
Copy link
Collaborator

@humberto-garza humberto-garza commented Jan 11, 2022

Description

When looking at the agent page, there is no way to tell if it is enabled or not or if it is online or not.

Implementation

Add a watcher to check for the agent page and when it finds the agent title row, query the agent status and add it to the view.

Testing

Before

image

After

Agent that is online and enabled
image

Agent that is online and disabled
image

Agent that is offline and enabled
image

Agent that is offline and disabled
image

@alejandro5042
Copy link
Owner

@humberto-garza: Still interested in this PR? You have a few build failures to fix then we can enable it:

Error:   926:18  error  'watchForAgentPage' is already defined      no-redeclare
Error:   951:62  error  ["team"] is better written in dot notation  dot-notation
Error:   953:56  error  ["pool"] is better written in dot notation  dot-notation
Error:   956:7   error  Expected space(s) after "if"                keyword-spacing
Error:   956:[28](https://github.com/alejandro5042/azdo-userscripts/actions/runs/3579620566/jobs/6020997280#step:4:29)  error  Expected '===' and instead saw '=='         eqeqeq
Error:   957:45  error  Strings must use singlequote                quotes
Error:   957:55  error  Operator ':' must be spaced                 space-infix-ops
Error:   957:56  error  Strings must use singlequote                quotes

@alejandro5042
Copy link
Owner

@humberto-garza - Is this still a desired PR?

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