Skip to content
This repository has been archived by the owner on Jan 24, 2022. It is now read-only.

Fix push functionality when deploying a dependency contract #1486

Merged
merged 3 commits into from
Feb 29, 2020

Conversation

frangio
Copy link
Contributor

@frangio frangio commented Feb 28, 2020

Fixes #1481. This bug had been introduced in #1426.

Should be cherry-picked into 2.8 release branch once merged.

@frangio frangio requested a review from ylv-io February 28, 2020 20:44
@frangio frangio added the status:ready-to-merge Order mergify to merge label Feb 28, 2020
Copy link
Contributor

@ylv-io ylv-io left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would be nice to add a test to cover that case.

@ylv-io ylv-io merged commit abe30da into OpenZeppelin:master Feb 29, 2020
@frangio
Copy link
Contributor Author

frangio commented Feb 29, 2020

I did think about a test but this error happens at a layer that isn't currently tested I think.

frangio added a commit that referenced this pull request Mar 2, 2020
* filter dependency contracts from alias list in push

* avoid mapping if variable is undefined

(cherry picked from commit abe30da)
@frangio frangio deleted the fix-create-from-dependency branch March 9, 2020 18:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status:ready-to-merge Order mergify to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Contract undefined not found with npx oz deploy of @openzeppelin/contracts-ethereum-package/StandaloneERC20
2 participants