Browse Source

Merge pull request #6579 from nocodb/nc-fix/exclude-query-from-error-response

refactor: Replace console.log with Nestjs Logger
pull/6580/head
mertmit 1 year ago committed by GitHub
parent
commit
f9100366ab
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 8
      packages/nocodb/src/services/datas.service.ts

8
packages/nocodb/src/services/datas.service.ts

@ -1,4 +1,4 @@
import { Injectable } from '@nestjs/common';
import { Injectable, Logger } from '@nestjs/common';
import { isSystemColumn } from 'nocodb-sdk';
import * as XLSX from 'xlsx';
import papaparse from 'papaparse';
@ -14,6 +14,8 @@ import NcConnectionMgrv2 from '~/utils/common/NcConnectionMgrv2';
@Injectable()
export class DatasService {
protected logger = new Logger(DatasService.name);
constructor() {}
async dataList(
@ -166,7 +168,7 @@ export class DatasService {
listArgs,
);
} catch (e) {
console.log(e);
this.logger.log(e);
NcError.internalServerError(
'Please check server log for more details',
);
@ -689,7 +691,7 @@ export class DatasService {
dependencyFields,
);
} catch (e) {
console.log(e);
this.logger.error(e);
NcError.internalServerError('Please check server log for more details');
}
}

Loading…
Cancel
Save