Remove the Keystore API #3657
Remove the Keystore API #3657
Conversation
StephenButtolph
requested review from
marun and
meaghanfitzgerald
as code owners
There was a problem hiding this comment.
@meaghanfitzgerald Is there a change that needs to be made to the docs repo with the removal of this file?
There was a problem hiding this comment.
The keystore API has been marked deprecated in the docs since March 2023. Once this PR is merged, we should include "[DEPRECATED]" in the title and sidebar, and remove the documentation entirely after about 4-6 weeks. cc: @owenwahlgren @ashucoder9
There was a problem hiding this comment.
link to docs: https://docs.avax.network/api-reference/keystore-api
There was a problem hiding this comment.
removing the Keystore API from docs, and marked as deprecated for those who view w direct link. changes should be live once the builders hub pr is live.
|
cc: @Andyvargtz |
|
Congrats on finally being able to remove this API lol 🥳 |
Why this should be merged
The Keystore API has been deprecated for almost 2 years. Since Etna was recently activated, now is the ideal time to remove it.
How this works
This PR removes all code related to the keystore.
How this was tested
CI
Need to be documented in RELEASES.md?
Yes.