openzeppelin-contracts-upgradeable

openzeppelin-contracts-upgradeable copied to clipboard

`initializer` modifier can't be nested, either. So it's not a difference. The comment of `initializer` says: > > * Similar to `reinitializer(1)`, except that in the context of a constructor...

zhiqiangxu

What's wrong? You have open low severity vulnerabilities. Github Vulnerability npm-undici/CVE-2024-24758 How to fix? 1. Visit the Vulnerabilities page to learn more about the unresolved vulnerabilities. 2. Remediate or deactivate...

vanta-with-task-management[bot]

https://github.com/OpenZeppelin/openzeppelin-contracts-upgradeable/blob/0b2bfd74c98d45143cc060417922b3c1328a9fef/contracts/finance/VestingWalletUpgradeable.sol#L53-L59 Docstring states that there is an owner and a pending owner although no pending owner is present. Moreover, the sender isn't necessarily set as owner/beneficiary. More accurate docstring would...

asardon

Reusing logic extracted to lib in this PR: https://github.com/OpenZeppelin/openzeppelin-contracts/pull/4799 #### PR Checklist - [ ] Tests - [ ] Documentation - [ ] Changeset entry (run `npx changeset add`)

k06a

**🧐 Motivation** There is currently no standard or well documented best practices for a use case where upgradeable contracts require chaining initializers as the contract develops. When a contract gets...

brunocangs

Improve Contracts API docs to function as Upgradeable docs too

2

The API docs for Contracts need some changes to make it clear that they are also the API docs for Contracts Upgradeable. It's only two things that are different: the...

frangio

1. Fixed the typo `trackedTransferedAmount` to `trackedTransferredAmount` to maintain consistency in variable names. 2. Corrected spelling errors in code comments for clarity and professionalism.

Pronoss

Hello While looking through your docs I found and fixed several spelling issues. Br, Elias.

nnsW3

This pull request addresses minor typographical errors in the `GUIDELINES.md` and `SECURITY.md` files: - In `GUIDELINES.md`, corrected the spelling of "mission-critical." - In `SECURITY.md`, replaced "on-going" with the correct form...

Dimitrolito

This PR removes a duplicated validity check in `ERC2771ForwarderUpgradeable._execute`, making the control flow easier to follow and reducing the risk of the conditions diverging over time.

sashaodessa