Inspired to check our own code to ensure we were always using forceApprove where necessary, and it seems like we missed one. (Probably redundant in this case, as the approval would almost certainly be 0 to start with, but doesn't hurt.)
Good thing this is the VaultAdmin - seems to have added 500 bytes?!
Type of change
[X] Bug fix
[ ] New feature
[ ] Breaking change
[ ] Dependency changes
[ ] Code refactor / cleanup
[ ] Optimization: [ ] gas / [ ] bytecode
[ ] Documentation or wording changes
[ ] Other
Checklist:
[X] The diff is legible and has no extraneous changes
[N/A] Complex code has been commented, including external interfaces
[ ] Tests have 100% code coverage
[X] The base branch is either main, or there's a description of how to merge
Description
Inspired to check our own code to ensure we were always using
forceApprove
where necessary, and it seems like we missed one. (Probably redundant in this case, as the approval would almost certainly be 0 to start with, but doesn't hurt.)Good thing this is the VaultAdmin - seems to have added 500 bytes?!
Type of change
Checklist:
main
, or there's a description of how to mergeIssue Resolution