Skip to content

Commit

Permalink
Merge pull request #172 from Brushfam/develop
Browse files Browse the repository at this point in the history
  • Loading branch information
Artemka374 authored Sep 25, 2023
2 parents 20ccd29 + 027b066 commit 49b09b3
Showing 1 changed file with 32 additions and 1 deletion.
33 changes: 32 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -27,4 +27,35 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

### Fixed
- Fixed reentrancy guard problem: [#88](https://github.com/Brushfam/openbrush-contracts/pull/88)
- Updated reentrancy example: [#108](https://github.com/Brushfam/openbrush-contracts/pull/108)
- Updated reentrancy example: [#108](https://github.com/Brushfam/openbrush-contracts/pull/108)

## [v4.0.0-beta.1]
## Changes

### Added
- Added new Governance feature with extensions: [#126](https://github.com/Brushfam/openbrush-contracts/pull/126) - was not under audit
- Added standard interface detection implementation: [#112](https://github.com/Brushfam/openbrush-contracts/pull/112) - was not under audit
- Added PSP22 Permit extension: [#109](https://github.com/Brushfam/openbrush-contracts/pull/109) - was not under audit
- Crypto module with some useful methods for better DX: [#109](https://github.com/Brushfam/openbrush-contracts/pull/109) - was not under audit

### Changes
- Bumped ink! version to 4.3.0: [#129](https://github.com/Brushfam/openbrush-contracts/pull/129)

## [v4.0.0]
## Changes

### Changed
- [*BREAKING*] Separated `_before_token_transfer` and `_after_token_transfer` in PSP22 to a trait `PSP22Transfer` and added
different implementation on whether `Capped` extension is implemented: [#141](https://github.com/Brushfam/openbrush-contracts/pull/141)
- [*BREAKING*] Removed `PSP22::approve` method, because it creates a double-spending issue. We recommend using `increase_allowance` and `decrease_allowance` instead:
[#138](https://github.com/Brushfam/openbrush-contracts/pull/138)
- [*BREAKING*] Changed errors to `isNotSet` from `isZeroAddress`: [#136](https://github.com/Brushfam/openbrush-contracts/pull/136)
- Added verification for `PSP22Wrapper` for whether the account in `deposit_for` and `withdraw_to` is underlying wrapper: [#140](https://github.com/Brushfam/openbrush-contracts/pull/140)
- Added `max_supply` internal method in `PSP22`, fixed the issue that amount can be calculated wrong way when using `Capped` and `FlashLender` together: [#142](https://github.com/Brushfam/openbrush-contracts/pull/142)
- `PaymentSplitter` now emits `PaymentReceived` event in `receive` method: [#139](https://github.com/Brushfam/openbrush-contracts/pull/139)
- `_release_all` method removed from `PaymentSplitter`, since it was considered error-prone and unnecessary: [#145](https://github.com/Brushfam/openbrush-contracts/pull/145)
- Added `_flash_fee_receiver` method to `FlashLender`, fee is now sent to beneficiary instead of being burned: [#157](https://github.com/Brushfam/openbrush-contracts/pull/157)
- Added `releasable` method to `PaymentSplitter`: [#146](https://github.com/Brushfam/openbrush-contracts/pull/146)
- [*BREAKING*] Ownership now can't be transferred to `None`: [#137](https://github.com/Brushfam/openbrush-contracts/pull/137)
- Fixed validation in `macro_definition`: [#144](https://github.com/Brushfam/openbrush-contracts/pull/144)
- Use boolean values in `reentrancy_guard`: [#143](https://github.com/Brushfam/openbrush-contracts/pull/143)

0 comments on commit 49b09b3

Please sign in to comment.