Differential D12111
[Cashtab] [Chronik] [Tx Gen] generateTokenTxOutput - part 1 token genesis tx Authored by emack on Oct 1 2022, 03:54. Tags None Subscribers None
Details
See T2503 for the pseudocode behind the generateTokenTxOutput function and how it'll be used by overarching token functions. Part 1 caters for the genesis token tx where the relevant OP_RETURN script and token/xec outputs are added to the transaction builder. Subsequent diff in this stack will cover the SEND/BURN OP_RETURN scripts and the additional outputs for token send/burn and token change. npm test and ensure the genesis unit test passes
Diff Detail
Event Timeline
Comment Actions
Comment Actions Build Bitcoin ABC Diffs / Diff Testing (cashtab-tests) failed.
Tail of the build log: npm WARN EBADENGINE Unsupported engine {
npm WARN EBADENGINE package: 'postcss-place@7.0.5',
npm WARN EBADENGINE required: { node: '^12 || ^14 || >=16' },
npm WARN EBADENGINE current: { node: 'v15.14.0', npm: '7.7.6' }
npm WARN EBADENGINE }
npm WARN EBADENGINE Unsupported engine {
npm WARN EBADENGINE package: 'postcss-preset-env@7.8.2',
npm WARN EBADENGINE required: { node: '^12 || ^14 || >=16' },
npm WARN EBADENGINE current: { node: 'v15.14.0', npm: '7.7.6' }
npm WARN EBADENGINE }
npm WARN EBADENGINE Unsupported engine {
npm WARN EBADENGINE package: 'postcss-pseudo-class-any-link@7.1.6',
npm WARN EBADENGINE required: { node: '^12 || ^14 || >=16' },
npm WARN EBADENGINE current: { node: 'v15.14.0', npm: '7.7.6' }
npm WARN EBADENGINE }
npm WARN EBADENGINE Unsupported engine {
npm WARN EBADENGINE package: 'postcss-selector-not@6.0.1',
npm WARN EBADENGINE required: { node: '^12 || ^14 || >=16' },
npm WARN EBADENGINE current: { node: 'v15.14.0', npm: '7.7.6' }
npm WARN EBADENGINE }
npm WARN deprecated stable@0.1.8: Modern JS already guarantees Array#sort() is a stable sort, so this library is deprecated. See the compatibility table on MDN: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/sort#browser_compatibility
npm WARN deprecated text-encoding@0.6.4: no longer maintained
npm WARN deprecated ts-custom-error@2.2.2: npm package tarball contains useless codeclimate-reporter binary, please update to version 3.1.1. See https://github.com/adriengibrat/ts-custom-error/issues/32
> cashtab@1.0.0 prepare
> cd ../.. && husky install web/cashtab/.husky
husky - Git hooks installed
added 1807 packages, and audited 1808 packages in 27s
232 packages are looking for funding
run `npm fund` for details
5 moderate severity vulnerabilities
To address all issues (including breaking changes), run:
npm audit fix --force
Run `npm audit` for details.
npm notice
npm notice New major version of npm available! 7.7.6 -> 8.19.2
npm notice Changelog: <https://github.com/npm/cli/releases/tag/v8.19.2>
npm notice Run `npm install -g npm@8.19.2` to update!
npm notice
> cashtab@1.0.0 build
> node scripts/build.js
Creating an optimized production build...
Failed to compile.
[eslint]
src/utils/cashMethods.js
Line 48:5: 'totalTokenInputUtxoValue' is assigned a value but never used no-unused-vars
Search for the keywords to learn more about each error.
Build cashtab-tests failed with exit code 1Comment Actions Default value for the totalTokenInputUtxoValue will be added in the next diff to avoid the '... varaible is defined but not used' build errors. Comment Actions Provide it with some kind of placeholder value here, e.g. true or false and then console.log it to remove this build error. I'm not sure if we can have a stacked diff land if one part isn't building.
Comment Actions
| |||||||||||||||||||||||