DonWillems
DonWillems
Hi @mihaelaaleks , do you have a status update?
Hi Michaela, it took a while for me to find the time, but here are some points that stood out to me: ### Bugs / Incorrect Behavior - [ ]...
- [ ] Replace DeliveredOn column with Linked Users column. In practice the table only shows the assets the messages are sent to (needs triple checking). This means the Target...
First of all, great work! Some things that stood out at first glance: - [ ] While sliding the bottom bar with only one datapoint (~now) this error is shown...
- Locally, when I view the history chart of an asset and resize the screen or when I switch to a different asset, I get this message: ``` index.ts:520 Uncaught...
Thanks for the changes! - I can't reproduce the first point anymore either. - I've noticed you left in a console log message 'onZoomChange' when you change the timeframe. -...
Need to do some more testing on an instance with more data. But this stood out to me when testing locally: https://github.com/user-attachments/assets/f170a849-5bab-409b-937d-411cad6a9a28 When first selecting an attribute in the history...
Did some more testing, looks good to me. With more datapoints the issue I reported above does not occur anymore. I think its because there is no line to draw...
As discussed, pattern matching the subscriptions set in a agent link to see if it is covered by the wildcard seems the best option. Keep in mind that a user...
> Notification language options don't belong in the manager_config....this is intended for the manager UI app notifications are a backend thing not related to the manager UI app. > >...