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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: prevent address reuse on Jam page #433

Merged
merged 2 commits into from
Aug 1, 2022

Conversation

theborakompanioni
Copy link
Collaborator

Add a similar check for reused addresses on the Jam page as it already exists on the Send page.

Before this PR, only duplicate address entries in the form inputs have been checked.
After this PR is applied, address validation is extended to all addresses already known to Jam.

馃摳 New, but duplicate address provided

馃摳 Already reused address provided

@theborakompanioni theborakompanioni added the bug Something isn't working label Jul 29, 2022
@theborakompanioni theborakompanioni self-assigned this Jul 29, 2022
@theborakompanioni theborakompanioni marked this pull request as ready for review July 29, 2022 10:55
@theborakompanioni theborakompanioni requested a review from a user July 29, 2022 10:55
@theborakompanioni theborakompanioni added enhancement New feature or request and removed bug Something isn't working labels Jul 29, 2022
@theborakompanioni theborakompanioni changed the title fix: prevent address reuse on Jam page feat: prevent address reuse on Jam page Jul 29, 2022
Copy link
Contributor

@dergigi dergigi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! Looks good to me.

I'd err on the side of brevity when it comes to wording, but that's just me. I plan to go over all strings before v0.1 anyway, so let's keep the wording as is for now.

@theborakompanioni theborakompanioni merged commit 6a8830f into master Aug 1, 2022
@theborakompanioni theborakompanioni deleted the prevent-address-reuse-on-jam-page branch August 1, 2022 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants