Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: get proxy #118

Merged
merged 1 commit into from
Jun 28, 2023
Merged

feat: get proxy #118

merged 1 commit into from
Jun 28, 2023

Conversation

PaulRBerg
Copy link
Owner

Closes #116.

@PaulRBerg PaulRBerg requested a review from andreivladbrg June 17, 2023 17:48
@PaulRBerg PaulRBerg changed the base branch from main to refactor/ownersip-redesign June 17, 2023 17:48
@PaulRBerg PaulRBerg force-pushed the refactor/ownersip-redesign branch from a15da64 to 33c9208 Compare June 17, 2023 18:27
@PaulRBerg PaulRBerg force-pushed the feat/get-proxy branch 3 times, most recently from b82eae6 to 036ba4a Compare June 17, 2023 18:46
@PaulRBerg PaulRBerg changed the base branch from refactor/ownersip-redesign to refactor/ownership-redesign June 17, 2023 18:46
@PaulRBerg PaulRBerg force-pushed the refactor/ownership-redesign branch from d457088 to 342ca1d Compare June 17, 2023 18:47
@PaulRBerg PaulRBerg force-pushed the feat/get-proxy branch 2 times, most recently from c206840 to 4a52570 Compare June 17, 2023 18:49
@PaulRBerg PaulRBerg force-pushed the refactor/ownership-redesign branch from 342ca1d to f5fa428 Compare June 17, 2023 19:36
@PaulRBerg PaulRBerg force-pushed the refactor/ownership-redesign branch from f5fa428 to 8e42354 Compare June 17, 2023 19:50
@PaulRBerg PaulRBerg force-pushed the refactor/ownership-redesign branch from 8e42354 to 9750486 Compare June 17, 2023 20:26
@PaulRBerg PaulRBerg force-pushed the refactor/ownership-redesign branch from 9750486 to c738a35 Compare June 18, 2023 09:59
Copy link
Collaborator

@andreivladbrg andreivladbrg left a comment

Choose a reason for hiding this comment

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

I think you missed this:

/// @dev See the documentation for the public functions that call this internal function.

User facing functions?

@PaulRBerg
Copy link
Owner Author

PaulRBerg commented Jun 18, 2023

Good catch!!

Resolved.

@PaulRBerg PaulRBerg force-pushed the refactor/ownership-redesign branch from c738a35 to b9144a7 Compare June 18, 2023 10:05
@PaulRBerg PaulRBerg force-pushed the refactor/ownership-redesign branch 2 times, most recently from b229563 to 17f65af Compare June 19, 2023 11:50
@PaulRBerg PaulRBerg force-pushed the refactor/ownership-redesign branch from af7353c to e060114 Compare June 23, 2023 09:05
@PaulRBerg PaulRBerg force-pushed the refactor/ownership-redesign branch from e060114 to 07fed42 Compare June 23, 2023 12:09
refactor: make proxies mapping internal
Base automatically changed from refactor/ownership-redesign to main June 28, 2023 18:50
@PaulRBerg PaulRBerg merged commit a2723b7 into main Jun 28, 2023
@PaulRBerg PaulRBerg deleted the feat/get-proxy branch June 29, 2023 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Provide "getProxy" getter for maximum clarity
2 participants