Make StarSystem required, remove StarPos as per discussion

This commit is contained in:
Jeremy Rimpo committed 2025-11-09 15:38:30 -06:00
1 parent 0e0d5db263
commit 8be89a51d9
2 files changed
+4 -13

No files matched your search

-7
View File
@@ -54,13 +54,6 @@ CarrierJump as appropriate.
**You MUST apply a location cross-check, as per
[Other data augmentations](../docs/Developers.md#other-data-augmentations).**
#### StarPos
You MUST add a `StarPos` array containing the system co-ordinates from the
last `FSDJump`, `CarrierJump`, or `Location` event.
**You MUST apply a location cross-check, as per
[Other data augmentations](../docs/Developers.md#other-data-augmentations).**
#### BodyID and BodyName
BodyID is already present in the form of the 'Body' key. This should be
renamed to BodyID to mirror many other events.
+4 -6
View File
@@ -33,7 +33,7 @@
"type" : "object",
"description" : "Contains all properties from the listed events in the client's journal minus the Localised strings and the properties marked below as 'disallowed'",
"additionalProperties" : false,
"required" : [ "timestamp", "event", "ScanType", "Genus", "Species", "SystemAddress", "BodyID"],
"required" : [ "timestamp", "event", "StarSystem", "ScanType", "Genus", "Species", "SystemAddress", "BodyID"],
"properties" : {
"timestamp": {
"type" : "string",
@@ -50,11 +50,9 @@
"type" : "boolean",
"description" : "Boolean value copied from the Journal LoadGame event, when it is present there."
},
"StarPos": {
"type" : "array",
"items" : { "type": "number" },
"minItems" : 3,
"maxItems" : 3,
"StarSystem": {
"type" : "string",
"minLength" : 1,
"description" : "Must be added by the sender"
},
"ScanType": {