Apply suggestions from code review

Typo and remove unnecessary parameter

Co-authored-by: Elsa Perelli <[email protected]>

src/services/export.service.js

@@ -31,7 +31,7 @@ const bulkExport = async (request, reply) => {
   }
   if (validateExportParams(parameters, reply)) {
     request.log.info('Base >>> $export');
-    const clientEntry = await addPendingBulkExportRequest(parameters._bySubject === 'Patient');
+    const clientEntry = await addPendingBulkExportRequest();
 
     const types = request.query._type?.split(',') || parameters._type?.split(','); //TODO, does gatherParams not pull from the query as well? Why is this OR required?
 

src/util/mongo.controller.js

@@ -109,7 +109,7 @@ const findResourcesWithAggregation = async (query, resourceType, options = {}) =
 /**
  * Called as a result of export request. Adds a new clientId to db
  * which can be queried to get updates on the status of the bulk export
- * @param {boolean} byPatient indicated whether this export request groups data by patient (versus by type)
+ * @param {boolean} byPatient indicates whether this export request groups data by patient (versus by type)
  * @returns the id of the inserted client
  */
 const addPendingBulkExportRequest = async (byPatient = false) => {