Skip to content
This repository has been archived by the owner on Jan 16, 2025. It is now read-only.

Rename Public/SecretKey into Note-Public/SecretKey and Public/SecretSpendKey into Public/SecretKey #77

Open
ureeves opened this issue Dec 20, 2022 · 1 comment
Labels
team:Core Low Level Core Development Team (Rust) team:research type:tech-debt the issue is a tech debt that needs fixing

Comments

@ureeves
Copy link
Member

ureeves commented Dec 20, 2022

Summary

We want to do a rename of the PublicKey and SecretKey structures to NotePublicKey and NoteSecretKey, respectively, as well as rename the PublicSpendKey and SecretSpendKey to PublicKey and SecretKey, respectively.
This is to better reflect their usage in our transaction model, and avoid any confusion when comparing documentation with this code.

Possible solution design or implementation

N/A

Additional context

The ideal situation would be that this repository would be independent of phoenix. This is, however, already not the case, since we use words like "Spend" and offer functionality such as SecretSpendKey::sk_r, which are clearly designed for use by phoenix.

@ureeves ureeves added type:tech-debt the issue is a tech debt that needs fixing team:Core Low Level Core Development Team (Rust) team:research labels Dec 20, 2022
@ureeves ureeves changed the title Rename Public/SecretKey into Note-Public/SecretKey and Public/SecretSpendKey into Public/SecretKey Rename Public/SecretKey into Note-Public/SecretKey and Public/SecretSpendKey into Public/SecretKey Dec 20, 2022
@javiersilva0
Copy link

See this document for more details.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
team:Core Low Level Core Development Team (Rust) team:research type:tech-debt the issue is a tech debt that needs fixing
Projects
None yet
Development

No branches or pull requests

2 participants