-
Notifications
You must be signed in to change notification settings - Fork 3
Conversation
There are lots of place still Nodary naming populated here some examples:
Also in my opinion variables named as |
I see thanks, I will go through the rest of the alerts as well then 😄 |
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.
Looks fine 😆 but related to Bedirhan's comment it's probably safe to do a project-wide replace of nodary with maybe "source" 🤷 ?
We could almost do this, there are a few db column names still containing nodary, but that should be ok. |
I'll try and deploy this and see what happens, can always revert. (update) Deployed... now we wait ~a day. |
…to update-missing-value-alert
No errors and/or issues, so this is running nicely. |
Closes [Airseeker Telemetry]: Swap out/remove error message Nodary reference#500