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

Rename removeOwner to removeOwnerWithThreshold #119

Closed
wants to merge 1 commit into from
Closed

Conversation

Uxio0
Copy link
Member

@Uxio0 Uxio0 commented Aug 8, 2019

Closes #118

@Uxio0 Uxio0 requested a review from rmeissner August 8, 2019 16:11
@Uxio0 Uxio0 self-assigned this Aug 8, 2019
@Uxio0 Uxio0 added this to the contracts-safe-1.1.0 milestone Aug 8, 2019
@rmeissner rmeissner removed this from the contracts-safe-1.1.0 milestone Sep 5, 2019
@rmeissner
Copy link
Member

I will close this for now, since we do not want to have signature changes at this point in time (issue stays open)

@rmeissner rmeissner closed this Sep 5, 2019
@rmeissner rmeissner deleted the issue-118 branch December 5, 2019 10:28
fdarian pushed a commit to fdarian/safe-contracts that referenced this pull request Jan 14, 2024
Add getAllTransactions endpoint to Safe Service Client
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.

Naming of functions for adding/removing owners is not coherent
2 participants