Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

Store address of Deposit Security Module inside Lido #435

Closed
ujenjt opened this issue Jun 14, 2022 · 5 comments · Fixed by #482
Closed

Store address of Deposit Security Module inside Lido #435

ujenjt opened this issue Jun 14, 2022 · 5 comments · Fixed by #482
Labels
next-upgrade Things to pickup for the next protocol upgrade

Comments

@ujenjt
Copy link
Member

ujenjt commented Jun 14, 2022

  • remove permission support for deposit which is potentially would allow deposits for several entities
  • add view method for fetch DSM for tooling needs
@TheDZhon TheDZhon added the next-upgrade Things to pickup for the next protocol upgrade label Jun 15, 2022
@TheDZhon TheDZhon mentioned this issue Jan 31, 2023
Merged
@TheDZhon TheDZhon linked a pull request Jan 31, 2023 that will close this issue
Shapella upgrade (capella+shanghai) #482
Merged
@TheDZhon
Copy link
Contributor

TheDZhon commented Mar 19, 2023 •
edited
Loading

Lido V2 introduced a swiss-knife tool to implement contracts discovery in one fell swoop: LidoLocator

@TheDZhon
Copy link
Contributor

TheDZhon commented Apr 6, 2023

Decided to re-open because V2 isn't deployed on mainnet yet.

@TheDZhon TheDZhon reopened this Apr 6, 2023
@0xMySt1c
Copy link

some what of a tangential question, but is it appropriate to use the lidolocator in smart contract implementation? or should i integrate using the steth proxy address.

@TheDZhon
Copy link
Contributor

Hey @0xMySt1c
Both options are good, would prefer to use LidoLocator though

@TheDZhon
Copy link
Contributor

TheDZhon commented May 26, 2023 •
edited
Loading

Fixed in https://github.com/lidofinance/lido-dao/releases/tag/v2.0.0

# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
next-upgrade Things to pickup for the next protocol upgrade
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Shapella upgrade (capella+shanghai) lidofinance/core
5 participants
@TheDZhon @ujenjt @0xMySt1c and others