Merge pull request #6496 from TheThingsNetwork/fix/sdk-todo-comment

Update SDK todo comment
TheThingsNetwork · Sep 4, 2023 · 91f67d5 · 91f67d5
2 parents 57b02bf + e24deae
commit 91f67d5
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/sdk/js/src/service/devices/merge.js b/sdk/js/src/service/devices/merge.js
@@ -65,7 +65,9 @@ export default (parts, base = {}, minimum = [['ids'], ['created_at'], ['updated_
             }
 
             if (this.isLeaf) {
-              // TODO: Remove this once https://github.com/TheThingsNetwork/lorawan-stack/issues/4766 is resolved.
+              // TODO: Instead of ignoring empty time values, the merge order should be improved
+              // as described in the following issue comment:
+              // https://github.com/TheThingsNetwork/lorawan-stack/issues/4766#issuecomment-952691708
               if (this.key.endsWith('_at') && e === '0001-01-01T00:00:00Z') {
                 return
               }