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

Request to merge small change to the windows file monitor #49

Merged
merged 7 commits into from
Nov 8, 2022

Conversation

AndreasB-atea
Copy link
Contributor

Fixed issue with tuncated alert message in the file monitor. Now it is correctly limited to 1024 in length

@stegenfeldt stegenfeldt self-assigned this Nov 8, 2022
@stegenfeldt
Copy link
Owner

Looks ok.
Build properly in VS2017.
I updated the changelog to include this fix

@stegenfeldt stegenfeldt merged commit b622908 into stegenfeldt:develop Nov 8, 2022
stegenfeldt added a commit that referenced this pull request Jan 31, 2023
* Added parameter UseServiceDisplayName for service discovery.

* Changed space to tab in AteaWinSvcServiceDiscoveryDS.vbs.

* Truncate File list in File Age Monitors
Fixes #45

* Set a proper  discovery interval

* Updated error handling on Fileshare monitoring
Fixes issue #47

* Add parameter for number of files for alert in File Age Monitor
Fixes #44

* truncated description even harder

* new version

* Updated .gitignore.

* Added monitor for service exist.

* Request to merge small change to the windows file monitor  (#49)

* Fixed issue with tuncated alert message
* small formatting adjustment, realigning a few whitespaces
Co-authored-by: Samuel Tegenfeldt <samuel.tegenfeldt@atea.se>

* Stegenfeldt/issue51 (#52)

Updated build test pipeline

* ignoring vs2022 stuff

* compiles alright after merge with pr

* version bump to 1.0.3.400
- pr 46-53

* Changelog updated for 1.0.3.400 release

---------

Co-authored-by: svantegraden <svante@snigel.eu>
Co-authored-by: Svante Graden <svante.graden@atea.se>
Co-authored-by: AndreasB-atea <65216950+AndreasB-atea@users.noreply.github.com>
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