-
Notifications
You must be signed in to change notification settings - Fork 16
feat: Update contract #108
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
Changes from all commits
ed20ca6
5ade718
78b3ac8
abbbb54
4d8d4ce
5eb82fb
e571670
a24f981
aa26eb6
31a1e0d
1511e23
6656179
11fb1a7
9ad1778
14420c8
17b4a1b
19924b3
21d66ed
17f7a92
a493a4a
3bae44f
3b9c7b0
d325390
afe0a84
98963ab
9f3316f
ff26932
13b98cf
8c72601
1454889
ec10fb2
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1 +1 @@ | ||
| 92849 | ||
| 92792 |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1 +1 @@ | ||
| 60493 | ||
| 60481 |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1 +1 @@ | ||
| 84809 | ||
| 84799 |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1 +1 @@ | ||
| 668482 | ||
| 668908 |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1 +1 @@ | ||
| 430781 | ||
| 431745 |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1 +1 @@ | ||
| 631634 | ||
| 631643 |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1 +1 @@ | ||
| 668994 | ||
| 669418 |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1 +1 @@ | ||
| 348258 | ||
| 339579 |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -22,13 +22,13 @@ abstract contract Auth is IAuth { | |
| } | ||
|
|
||
| /// @inheritdoc IAuth | ||
| function rely(address user) external auth { | ||
| function rely(address user) public auth { | ||
| wards[user] = 1; | ||
| emit Rely(user); | ||
| } | ||
|
|
||
| /// @inheritdoc IAuth | ||
| function deny(address user) external auth { | ||
| function deny(address user) public auth { | ||
|
Comment on lines
+25
to
+31
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think these can be
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. No, the investment manager is still using them from
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Could you link me to that line?
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Ok, I see, later the vault needs to call manager 👍🏻. |
||
| wards[user] = 0; | ||
| emit Deny(user); | ||
| } | ||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.