Permalink
Comparing changes
Choose two branches to see what’s changed or to start a new pull request.
If you need to, you can also or
learn more about diff comparisons.
Open a pull request
Create a new pull request by comparing changes across two branches. If you need to, you can also .
Learn more about diff comparisons here.
...
- 6 commits
- 44 files changed
- 2 contributors
Commits on Sep 18, 2024
-
feat(logger): implement configurable logging system (#264)
* refactor: replace every console.log with console.info These will all be replaced with the debugging logger that will be introduced in this branch. * refactor(*): update logging system for wallet clients - Replaced `console.info`, `console.warn`, and `console.error` calls with appropriate logger methods (`this.logger.info`, `this.logger.warn`, `this.logger.error`, `this.logger.debug`) - Wrapped `signTransactions` method logic in a try/catch block to handle and log errors consistently - Ensured that all wallet clients leverage the scoped logger created in the `BaseWallet` class - Added more logging statements to provide better visibility into the wallets' operations * refactor(wallets): enhance signTransactions logging - Add detailed logging to `signTransactions` methods across wallet clients. - Include debug logs for transaction processing steps and results. * refactor(WalletManager): update createAlgodClient method This commit refactors the createAlgodClient method in the WalletManager class to improve its functionality and logging. The main changes are: - Update method signature to accept `networkId` instead of `config` - Move network config retrieval inside the method - Adjust logging to use the passed `networkId` parameter These changes should resolve issues related to accessing the active network property on initialization and improve the overall structure of the method. * test(wallets): update wallet tests to use mocked Logger This commit updates the wallet test files to use the new `Logger` system instead of `console` logging. The main changes include: - Update mock setup for logger in all wallet test files - Replace `console.warn` and `console.error` expectations with `logger.warn` and `logger.error` in relevant tests - Adjust test assertions to check for `logger` calls instead of `console` calls - Remove unused `console` mocks from test files * refactor(store): replace console with logger in store functions - Replace `console.warn` calls with `logger.warn` in store functions - Update tests to mock the `logger` to suppress output during test runs * test(logger): add unit tests for Logger class - Create new test file for `Logger` class - Implement tests for singleton instance, log levels, and scoped logging - Add `setIsClient` method to Logger for testing purposes - Ensure tests cover all logging scenarios and respect log levels
-
chore(deps): update non-major dependencies (#262)
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
-
chore(deps): pin dependencies (#261)
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
-
chore(deps): lock file maintenance (#263)
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
-
fix: correct DEBUG log level handling in WalletManager (#265)
- Refactor logger initialization in `WalletManager` to correctly handle DEBUG log level - Add `WalletManagerOptions` interface for better type safety - Improve log level determination logic - Export `WalletManagerOptions` from index.ts for external use This commit ensures that the DEBUG log level is properly set and respected, fixing an issue where DEBUG logs were not being output as expected.
-
Loading
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff v3.4.0...v3.5.0