fix: Wrap read only relationships deletion in try catch (#4056)

* fix: Event creation/deletion error

* try-catch

* fix final

* Adding comments

Co-Authored-By: Areeb Jamal <[email protected]>

* Trailing space fix

Co-authored-by: Areeb Jamal <[email protected]>

Author: kushthedude

app/serializers/event.js

@@ -13,10 +13,20 @@ export default ApplicationSerializer.extend(CustomPrimaryKeyMixin, {
 
   serialize() {
     const json = this._super(...arguments);
-    delete json.data.relationships['general-statistics'];
-    delete json.data.relationships['order-statistics'];
+
+    const { relationships } = json.data;
+    // We are deleting read only relationships so that they don't
+    // break the server. Since these relationships are not always
+    // present, we catch and ignore the error
+    try {
+      delete relationships['general-statistics'];
+    } catch {} // eslint-disable-line no-empty
+    try {
+      delete relationships['order-statistics'];
+    } catch {} // eslint-disable-line no-empty
 
     return json;
+
   }
 
 });