fix: order webpack plugins

src/models/transactions/common.ts

@@ -176,18 +176,6 @@ export interface BaseTransaction {
    * account it says it is from.
    */
   TxnSignature?: string
-  /**
-   * The date/time when this transaction was included in a validated ledger.
-   */
-  date?: number
-  /**
-   * An identifying hash value unique to this transaction, as a hex string.
-   */
-  hash?: string
-  /**
-   * The sequence number of the ledger that included this transaction.
-   */
-  ledger_index?: number
 }
 
 /**

test/webpack.config.js

@@ -34,12 +34,6 @@ function webpackForTest(testFileName) {
       filename: match[1] + '.js',
     },
     plugins: [
-      new webpack.ProvidePlugin({ process: 'process/browser' }),
-      new webpack.ProvidePlugin({ Buffer: ['buffer', 'Buffer'] }),
-      new webpack.IgnorePlugin({
-        resourceRegExp: /^\.\/wordlists\/(?!english)/,
-        contextRegExp: /bip39\/src$/,
-      }),
       // this is a bit of a hack to prevent 'bn.js' from being installed 6 times
       // TODO: any package that is updated to use bn.js 5.x needs to be removed from `bnJsReplaces` above
       // https://github.com/webpack/webpack/issues/5593#issuecomment-390356276
@@ -52,6 +46,12 @@ function webpackForTest(testFileName) {
           resource.request = 'diffie-hellman/node_modules/bn.js'
         }
       }),
+      new webpack.ProvidePlugin({ process: 'process/browser' }),
+      new webpack.ProvidePlugin({ Buffer: ['buffer', 'Buffer'] }),
+      new webpack.IgnorePlugin({
+        resourceRegExp: /^\.\/wordlists\/(?!english)/,
+        contextRegExp: /bip39\/src$/,
+      }),
     ],
     module: {
       rules: [

webpack.config.js

@@ -24,13 +24,6 @@ function getDefaultConfiguration() {
       filename: `xrpl.default.js`,
     },
     plugins: [
-      new webpack.NormalModuleReplacementPlugin(/^ws$/, './WSWrapper'),
-      new webpack.ProvidePlugin({ process: 'process/browser' }),
-      new webpack.ProvidePlugin({ Buffer: ['buffer', 'Buffer'] }),
-      new webpack.IgnorePlugin({
-        resourceRegExp: /^\.\/wordlists\/(?!english)/,
-        contextRegExp: /bip39\/src$/,
-      }),
       // this is a bit of a hack to prevent 'bn.js' from being installed 6 times
       // TODO: any package that is updated to use bn.js 5.x needs to be removed from `bnJsReplaces` above
       // https://github.com/webpack/webpack/issues/5593#issuecomment-390356276
@@ -43,6 +36,13 @@ function getDefaultConfiguration() {
           resource.request = 'diffie-hellman/node_modules/bn.js'
         }
       }),
+      new webpack.NormalModuleReplacementPlugin(/^ws$/, './WSWrapper'),
+      new webpack.ProvidePlugin({ process: 'process/browser' }),
+      new webpack.ProvidePlugin({ Buffer: ['buffer', 'Buffer'] }),
+      new webpack.IgnorePlugin({
+        resourceRegExp: /^\.\/wordlists\/(?!english)/,
+        contextRegExp: /bip39\/src$/,
+      }),
     ],
     module: {
       rules: [],