Skip to content

fix: v1.8.2-rc.3 qa issues#246

Merged
r4mmer merged 4 commits intorelease-candidatefrom
fix/1.8.2-rc.3-QA-issues
Apr 14, 2025
Merged

fix: v1.8.2-rc.3 qa issues#246
r4mmer merged 4 commits intorelease-candidatefrom
fix/1.8.2-rc.3-QA-issues

Conversation

@r4mmer
Copy link
Copy Markdown
Member

@r4mmer r4mmer commented Apr 11, 2025

Motivation

During the v1.8.2-rc.3 QA some issues were found.
By following the logs we found 2 relevant error logs:

  1. [hathor-wallet-service-mainnet-stg-createTxProposalApi] Unhandled error on /tx/proposal: TypeError: Cannot read properties of undefined (reading 'name')
  2. [hathor-wallet-service-mainnet-stg-getTxOutputs] Unhandled error on /wallet/tx_outputs: Error: Addresses can't be empty.

Acceptance Criteria

  • Fix issues found during the QA for version v1.8.2-rc.3

Checklist

  • If you are requesting a merge into master, confirm this code is production-ready and can be included in future releases as soon as it gets merged
  • Make sure either the unit tests and/or the QA tests are capable of testing the new features
  • Make sure you do not include new dependencies in the project unless strictly necessary and do not include dev-dependencies as production ones. More dependencies increase the possibility of one of them being hijacked and affecting us.

@r4mmer r4mmer self-assigned this Apr 11, 2025
@r4mmer r4mmer moved this from Todo to In Progress (Done) in Hathor Network Apr 14, 2025
Comment thread packages/wallet-service/tests/nodeConfig.test.ts Outdated
@github-project-automation github-project-automation Bot moved this from In Progress (Done) to In Review (WIP) in Hathor Network Apr 14, 2025
@r4mmer r4mmer moved this from In Review (WIP) to In Review (Done) in Hathor Network Apr 14, 2025
@r4mmer r4mmer merged commit 4d818ed into release-candidate Apr 14, 2025
1 check passed
@github-project-automation github-project-automation Bot moved this from In Review (Done) to Waiting to be deployed in Hathor Network Apr 14, 2025
@r4mmer r4mmer deleted the fix/1.8.2-rc.3-QA-issues branch April 14, 2025 18:29
@r4mmer r4mmer moved this from Waiting to be deployed to Done in Hathor Network Apr 30, 2025
r4mmer added a commit that referenced this pull request Jul 3, 2025
…-lib-v2.1.1

* origin/master:
  chore: bumped to v1.9.0 (#259)
  chore: nano-testnet-hackaton (#243)
  fix: address invalid parameter (#235)
  chore: added htr token seed (#256)
  fix: invalid assert in release tx proposal utxos (#251)
  fix: we should await mark utxos with proposal id to prevent a race condition (#250)
  Merge pull request #248 from HathorNetwork/feat/version-camel-case
  fix: v1.8.2-rc.3 qa issues (#246)
  fix: firebase env variables should be optional (#245)
  chore: remove error handler middleware from authorization lambda (#242)
  feat: version data validation (#241)
  fix: Mainnet release script (#203)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants