-
Notifications
You must be signed in to change notification settings - Fork 86
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
Contract size optimization - remove unused royalty fetcher to reduce contract size #140
Conversation
Current dependencies on/for this PR:
This comment was auto-generated by Graphite. |
76141cd
to
34ea083
Compare
34ea083
to
03750ae
Compare
d3dd40e
to
7d1c3de
Compare
03750ae
to
7d711db
Compare
7d1c3de
to
e786b8d
Compare
e786b8d
to
c3c8cca
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought we used this on the contract level but I now realized we removed it. Did you check the front-end usage?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mattlenz are you using the read method supplyRoyaltyInfo
anywhere in the frontend? I didn't see it anywhere when searching the repo
Search looks good: https://github.com/search?q=org%3Aourzora%20supplyRoyaltyInfo&type=code |
No description provided.