Skip to content
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

addpa command allows adding of duplicate public addresses #12

Open
Haris-Irfan opened this issue Nov 8, 2024 · 0 comments
Open

addpa command allows adding of duplicate public addresses #12

Haris-Irfan opened this issue Nov 8, 2024 · 0 comments

Comments

@Haris-Irfan
Copy link
Owner

Haris-Irfan commented Nov 8, 2024

Running addpa 4 c/BTC l/savings_wallet pa/0x71C7656EC7ab88b098defB751B7401B5f6d8976F and then running addpa 5 c/BTC l/savings_wallet pa/0x71C7656EC7ab88b098defB751B7401B5f6d8976F allows me to add the same public address to multiple contacts. I feel like a check could be put in place to ensure that duplicate public addresses are not used since two wallets cannot have the same public address!

Screenshot 2024-11-08 at 6.03.49 PM.png

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant