-
Notifications
You must be signed in to change notification settings - Fork 47
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 first version of events #71
Merged
Merged
Conversation
This file contains 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
…aasx-server into aorzelski/events
Increment it in timeseries thread loop Show it in blazor tree
Show TimeStamp in blazor tree
Path /diff/updates can show also values of properties
* first implementation with message envelope
* working with event element CreateUpdateDelete
* implement specified structure/ update value events * time-based delta seems not to be working
* try fix event sending of CREATE events
* pushing disabling of code piece
Show reference and timestamps of observed element
…x-server into MIHO/eventsv2
Format code to complete CI on github
* try fix event sending of CREATE events
* pushing disabling of code piece
* fix serialization
Show reference and timestamps of observed element
Format code to complete CI on github
Debug code added and commented out after bugfix
For /diff/values/searchPath show only values if searchPath is in path auto=seconds can be combined with parameter refresh to only show changes in auto=seconds
Debug code added and commented out after bugfix
Add missing libgidplus for qrcoder
Add missing libgdiplus for qrcoder
Libgdiplus can not be installed on docker for ARM 32. Fix later.
Build did noz work. remove last changes.
mmarseu
reviewed
Jun 7, 2021
@@ -3635,7 +3635,10 @@ public static void securityInit() | |||
var sm = env.AasEnv.FindSubmodel(smr); | |||
if (sm != null && sm.idShort != null) | |||
{ | |||
sm.SetAllParents(); | |||
if (!sm.idShort.ToLower().Contains("Security")) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't check if String.Contains()
has been overridden in this application but unless that's the case, this check doesn't work. It compares a string with a capital letter against a fully lower-cased idShort.
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.
No description provided.