refactor: removes 'orders' field from the ICommunity interface

The 'orders' field is not part of the Mongoose document, but rather it's something that's calculated on-the-fly
as part of the response to the `/findcomms` command for each community. Converting the mongoose document to a
plain object gets rid of the TSC errors without having to introduce an 'orders' field to the ICommunity interface.

This solution is preferred as the ICommunity interface should ideally mirror what's stored in the database.

bot/modules/community/commands.js

@@ -22,8 +22,9 @@ async function findCommunities(currency) {
   });
   const orderCount = await getOrderCountByCommunity();
   return communities.map(comm => {
-    comm.orders = orderCount[comm.id] || 0;
-    return comm;
+    const plainCommunity = comm.toObject();
+    plainCommunity.orders = orderCount[comm.id] || 0;
+    return plainCommunity;
   });
 }
 

models/community.ts

@@ -41,7 +41,6 @@ export interface ICommunity extends Document {
     order_channels: Types.DocumentArray<IOrderChannel>;
     fee: number;
     earnings: number;
-    orders: number;
     orders_to_redeem: number;
     dispute_channel: string;
     solvers: Types.DocumentArray<IUsernameId>;