Open
Conversation
…BIOSAssetTag from.
Owner
|
@shoustech wow, thank you! I will check it the upcoming days and if is also works here on my side, I will crate a new release. |
Removed unneeded if statements. Added a backup for when we do not recognize the monitor display technology.
Fixed an issue on mac systems where wifi.js was reporting an error when trying to list networks.
Author
|
I have been using this change on a few hundred PCs for about 3-4 months now and it has been working without issue. I am going to be updating my agent and will be updating my systeminformation dependency. Would you like an updated pull request? I also have code we are using outside of the module that would be good features if you wanted the pull requests.
|
|
@shoustech i am really interested in the code to find out if bitlocker is set up. i could not find it in your fork, where could i find the code? |
780d938 to
b319280
Compare
9d86227 to
b626c2a
Compare
Update wifi.js
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Note:
I am not using the previously used variable
ssectionsbecause I could not find a reliable way to map any of the displays to the values output. I can go back in and addpixelDepthandmainwith the value from the main display inssectionsas the default for all the monitors.Description
I am opening this pull request because after #853 was submitted I ended up looking at the commands being used to pull the information on windows and noticed that there was some room for improvement. I fixed an issue where in some situations the output would exclude displays. I have also added the fields
displayIdandproductionYearwhich was previously only available on MacOS.Notable changes
The number of displays is now based on the variable
tsections. I then match the InstanceName property to the Instance name property inmsectionsandisections. FordsectionsI am finding the correct display by matchingtsections.InstanceNametodsections.PNPDeviceIDAfter all of the objects associated with the display have been found the app will then pull variables in this order:
tsectiondsectionmsectionand finally
isectionSince the output of
dsectionis the most complete I pulled what I could from that object first and then went down the list from there. If a variable was not pulled fromdsectionor the correlated object could not be found I would then fill in blank values or pull backup variables from the other objects.Also:
displayIdandproductionYearExamples
In this situation I am using a RDP connection to a different desktop
Existing module
Modified module
Without RDP original code:
Without RDP modified code: