Skip to content

Commit

Permalink
[@dhealthdapps/backend] refactor(discovery): update debug option valu…
Browse files Browse the repository at this point in the history
…e of discoverTransactions scheduler to be false if app is in staging/production environment
  • Loading branch information
nsonanh committed Dec 27, 2022
1 parent 9426da1 commit 804b98c
Showing 1 changed file with 1 addition and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,6 @@ export interface DiscoverTransactionsCommandOptions
* scheduler. Contains source code for the execution logic of a
* command with name: `discovery:DiscoverTransactions`.
*
* @todo Should use `BigInt` in {@link extractTransactionBlock} because `height.compact()` is not protected for number overflow.
* @since v0.2.0
*/
@Injectable()
Expand Down Expand Up @@ -332,10 +331,9 @@ export class DiscoverTransactions extends DiscoveryCommand {

// executes the actual command logic (this will call discover())
// additionally, updates state.data.lastUsedAccount
// @todo remove debug flag for staging/production releases
await this.run(["both"], {
source,
debug: false,
debug: process.env.NODE_ENV === "development",
} as DiscoveryCommandOptions);
}

Expand Down Expand Up @@ -736,7 +734,6 @@ export class DiscoverTransactions extends DiscoveryCommand {
* The transaction header can always be re-created using the other fields
* present in the {@link Transaction} document.
*
* @todo Move to a bytes-optimized storage format for payloads (only message is necessary)
* @param {Transaction} transaction
* @returns {string}
*/
Expand Down

0 comments on commit 804b98c

Please sign in to comment.